Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: Upgrade to latest pulumi/pulumi #474

Merged
merged 2 commits into from
Aug 1, 2023
Merged

deps: Upgrade to latest pulumi/pulumi #474

merged 2 commits into from
Aug 1, 2023

Conversation

abhinav
Copy link
Contributor

@abhinav abhinav commented Aug 1, 2023

The version of Pulumi used by pulumi-yaml
has not been updated in a while.

Update to the latest release.

Copy link
Contributor Author

abhinav commented Aug 1, 2023

The version of Pulumi used by pulumi-yaml
has not been updated in a while.

Update to the latest release.
Gets the latest testdata with `make get_testdata`,
updates `get_schemas.sh` with schemas referenced by these tests,
and runs codegen to regenerate all cases.

Broken tests were opted out, with the following issues created for them.

- #476
- #477
- #478
- #479
- #480
@abhinav abhinav merged commit e98eb01 into main Aug 1, 2023
4 checks passed
@abhinav abhinav deleted the abhinav/pulumi-up branch August 1, 2023 22:13
abhinav added a commit that referenced this pull request Aug 1, 2023
Runs `go mod tidy` with -go=1.19 on the go.mod for the project.
Note that as of Go 1.20, the `go` directive is **not**
the minimum required Go version.
It's the version of the toolchain we developed against.
This is changing in Go 1.21.

The `-compat` flag specifies which version of Go the go.mod file
needs to be compatible with.
By default, this is one release behind the `-go` flag
(or `go` directive).
So we don't need to specify it manually.

Lastly, this deletes the `go mod tidy` in the test job.
Only the lint job should bbe modifying the go.mod
to ensure that the go.mod is up to date.

(This is a copy of #471, which was accidentally merged into #474.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants