-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
Terraform Provider v5 Support #93
Conversation
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.
Bridgecrew has found errors in this PR ⬇️
/terratest |
/terratest |
/terratest |
/terratest |
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.
Convert to pass-through to terraform-aws-s3-bucket so we can archive this module. s3-bucket should already have all the necessary features, and if not, add them there. LMK if it is too much work.
Update: it is, in fact, too much work.
We decided that we will leave this as a final PR for the module, and redirect users to use the |
/terratest |
Labeled as a major release to call attention to final PR |
/terratest |
This may be the final PR for this module. This module offers no value beyond the
s3-bucket
module and theterraform-aws-route53-alias
module, so Cloud Posse is phasing out support for this redundant module. However, this PR should leave this module in a workable state until archival.what
.github
configtflint
requirementswhy
public-read
ACL to satisfy new S3 requirementsreferences
Closes Related PRs
Closes Related Issues
context.tf
#75