-
Notifications
You must be signed in to change notification settings - Fork 249
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
Fix terraform extract #187
Conversation
I identified that I had set a variable to be constant rather than mutable. added a commit to fix that. Validated within a private GitHub workflow this morning that it functions as expected. (Well, it happens to be morning where I am right now) |
Thanks for the contribution @cpc-camarj! I'll be able to review this tomorrow, but it looks good so far. |
Apologies for the delay in merging. Thank you! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
On Windows runners, extracting the downloaded CLI zip file was failing because the file didn't have a
.zip
extension. This PR attempts to solve the problem by adding the extension to the downloaded file before extraction.(fixes #186)