Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update context provider versions #52

Merged
merged 6 commits into from
Nov 24, 2020
Merged

Update context provider versions #52

merged 6 commits into from
Nov 24, 2020

Conversation

joe-niland
Copy link
Member

what

  • use context.tf
  • bump upstream module versions
  • use lower bound provider pinning

why

  • update module in line with latest approaches

references

@joe-niland joe-niland requested a review from a team as a code owner November 17, 2020 01:07
@joe-niland joe-niland requested review from 3h4x and aknysh and removed request for a team November 17, 2020 01:07
@joe-niland
Copy link
Member Author

/test all

@joe-niland
Copy link
Member Author

/test all

@joe-niland
Copy link
Member Author

/test terratest

@joe-niland joe-niland requested a review from jamengual November 17, 2020 04:58
@joe-niland joe-niland merged commit 698bb1e into cloudposse:master Nov 24, 2020
@joe-niland joe-niland deleted the update-context-provider-versions branch November 24, 2020 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@osterman osterman osterman approved these changes

@3h4x 3h4x Awaiting requested review from 3h4x

@aknysh aknysh Awaiting requested review from aknysh

@jamengual jamengual Awaiting requested review from jamengual

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

update required_providers in all module
2 participants
@joe-niland @osterman