Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

minimum required Terraform version bumped to 0.13.0, context.tf updated, readme updated #37

Merged
merged 1 commit into from
Feb 4, 2021
Merged

minimum required Terraform version bumped to 0.13.0, context.tf updated, readme updated #37

merged 1 commit into from
Feb 4, 2021

Conversation

maximmi
Copy link
Contributor

@maximmi maximmi commented Feb 3, 2021

what

  • update context.tf to v0.23.0
  • minimum required Terraform version bumped to 0.13.0
  • readme updated, Bridgecrew compliance badges added

why

  • It allows for setting the letter case of tag names and labels
  • we have dropped support for Terraform 0.12
  • To be able see and fix the recommendations from Bridgecrew so we can position our modules as standards compliant

@maximmi maximmi requested review from a team as code owners February 3, 2021 17:37
@maximmi maximmi requested review from jhosteny and brcnblc and removed request for a team February 3, 2021 17:37
@maximmi maximmi self-assigned this Feb 3, 2021
@maximmi
Copy link
Contributor Author

maximmi commented Feb 3, 2021

/test all

@osterman osterman merged commit a9191d1 into master Feb 4, 2021
@osterman osterman deleted the required-tf-13 branch February 4, 2021 07:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@osterman osterman osterman approved these changes

@jhosteny jhosteny Awaiting requested review from jhosteny

@brcnblc brcnblc Awaiting requested review from brcnblc

Assignees

@maximmi maximmi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maximmi @osterman