-
Notifications
You must be signed in to change notification settings - Fork 151
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
Upgrade terraform-provider-aws to v4.60.0 #2440
Conversation
Does the PR have any schema changes?Looking good! No breaking changes found. New resources:
New functions:
|
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.
This all looks fine to me.
Only comment I'd make on your new framework reverts is that you'd deleted a whole file - this could cause conflicts so it might be worth switching to just excluding it from the build with an exclude build flag.
Does the PR have any schema changes?Looking good! No breaking changes found. New resources:
New functions:
|
1 similar comment
Does the PR have any schema changes?Looking good! No breaking changes found. New resources:
New functions:
|
Does the PR have any schema changes?Looking good! No breaking changes found. New resources:
New functions:
|
Does the PR have any schema changes?Looking good! No breaking changes found. New resources:
New functions:
|
In our fork, this upgrade required reverting the migration of
cognitoidp
to the plugin framework. I've tried to logically follow this PR but am unsure how to make sure the result is correct.resolves #2422