-
Notifications
You must be signed in to change notification settings - Fork 112
Added security_and_analysis block support #143
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
Added security_and_analysis block support #143
Conversation
b85c12f to
346b4e0
Compare
346b4e0 to
5b0ddf5
Compare
hisabimbola
commented
Mar 20, 2023
@mineiros-ci @mineiros-team any update on when this PR can be merged. It'll really be useful to have security configuration
gaima8
commented
Jun 15, 2023
As I understand it the container used to run checks on PR (make test/docker/pre-commit) has terraform v1.1.7.
This PR uses optional attributes with a default value. terraform didn't add support for optional attributes with a default value until v1.3 though.
https://developer.hashicorp.com/terraform/language/v1.1.x/expressions/type-constraints#experimental-optional-object-type-attributes
https://developer.hashicorp.com/terraform/language/expressions/type-constraints#optional-object-type-attributes
Among other things https://github.com/mineiros-io/build-tools/blob/422e2711d58426624e1b3db54eff8d1620682a2c/Makefile#L8 would need to change to support this.
No description provided.