-
-
Notifications
You must be signed in to change notification settings - Fork 104
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
remove unused hashicorp/local provider #66
Conversation
/terratest |
/terratest |
@Gowiem / @gberenice do I have to do something to get these CI checks to complete? |
/terratest |
@megashby as a security measure, you need to be within the contributor team. Otherwise, a brand new contributor could open a PR that creates 1000x cryptominers in the CP AWS account through Terratest and suddenly their bill is a million bucks. I've just kicked them off for you 👍 |
Thank you for this @Gowiem! If you could additionally provide guidance on if there is anything else the maintainers would like to see here it would be very appreciated! We really love using this module in our org but due to our new security requirements we are facing issues with the current state and it's inclusion of the unused hashicorp/local provider. |
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.
LGTM
These changes were released in v1.0.1. |
what
why
references