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

Change provider block to required provider #4

Merged
merged 3 commits into from
Feb 8, 2023
Merged

Change provider block to required provider #4

merged 3 commits into from
Feb 8, 2023

Conversation

dragonfleas
Copy link
Contributor

  • Update github workflows to our new standard
  • Update provider block to be a required provider instead of an explicit one due to a bug when trying to destroy modules with child providers listed
  • Remove region variable

@dragonfleas dragonfleas added the bug Something isn't working label Feb 8, 2023
@dragonfleas dragonfleas self-assigned this Feb 8, 2023
@dragonfleas dragonfleas merged commit 01a61dc into main Feb 8, 2023
@dragonfleas dragonfleas deleted the required-provider-fix branch February 8, 2023 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chrisshiplet chrisshiplet chrisshiplet approved these changes

@spruce-bruce spruce-bruce Awaiting requested review from spruce-bruce

@Mistweaver Mistweaver Awaiting requested review from Mistweaver

Assignees

@dragonfleas dragonfleas

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dragonfleas @chrisshiplet