-
Notifications
You must be signed in to change notification settings - Fork 80
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
Update go sdk v3 #1399
Update go sdk v3 #1399
Conversation
Does the PR have any schema changes?Looking good! No breaking changes found. |
Version computation is not right per pulumi/provider-version-action#13 |
06798f8
to
a7768c9
Compare
Tagging this PR as needs-release/major has resulted in |
A tad better but tests need to be updated to point to V3 now. |
) | ||
|
||
replace github.com/pulumi/pulumi-eks/sdk/v3 => ../../../sdk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be good to avoid keeping the replaces but I can't seem to be able to run go mod tidy
since there is no valid v3 SDK published yet. Trying replaces temporarily.
Go tests actually passed against the V3 SDK, that's good. |
I'm going to merge and cut another alpha. |
Go SDK must be versioned as v3 otherwise Go refuses to use this.