-
-
Notifications
You must be signed in to change notification settings - Fork 221
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
feat: Added support for deduplication_scope
and fifo_throughput_limit
#32
feat: Added support for deduplication_scope
and fifo_throughput_limit
#32
Conversation
input_deduplication_scope
input_fifo_throughput_limit
and fix exampleinput_deduplication_scope
and input_fifo_throughput_limit
input_deduplication_scope
and input_fifo_throughput_limit
deduplication_scope
and fifo_throughput_limit
a6ba7cb
to
edce090
Compare
@antonbabenko comments addressed, please review 🙏 |
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.
Please update versions.tf
to use Terraform 0.13.1 at least, and AWS provider when the feature you are adding was introduced.
Copy this - https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/master/versions.tf
Get account_id from aws_caller_identity
Good job! Here we go 🎉 v3.2.0 has been just released. |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
Motivation and Context
Fixes #31
Breaking Changes
None.
How Has This Been Tested?
examples/*
projects