-
Notifications
You must be signed in to change notification settings - Fork 33
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
Terraform 0.14 upgrade #31
Conversation
/test all |
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.
When you use expressions in an if conditional, you may omit the expression syntax (${{ }}) because GitHub automatically evaluates the if conditional as an expression. For more information about if conditionals, see Workflow syntax for GitHub Actions.
/test all |
/rebuild-readme |
/test all |
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!
.github/workflows/auto-context.yml
Outdated
@@ -34,7 +34,7 @@ jobs: | |||
fi | |||
|
|||
- name: Create Pull Request | |||
if: steps.update.outputs.create_pull_request == 'true' | |||
if: {{ steps.update.outputs.create_pull_request == 'true' }} |
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.
When you use expressions in an if conditional, you may omit the expression syntax (${{ }}) because GitHub automatically evaluates the if conditional as an expression. For more information about if conditionals, see Workflow syntax for GitHub Actions.
* make this module v14 compatible (cloudposse#27) Co-authored-by: actions-bot <[email protected]> * Terraform 0.14 upgrade (cloudposse#28) * Fix auto-context and validate-codeowners (cloudposse#29) * Update README.md and docs (cloudposse#30) Co-authored-by: Nuru <[email protected]> * Terraform 0.14 upgrade (cloudposse#31) * Terraform 0.14 upgrade * updated workflows * Updated README.md Co-authored-by: actions-bot <[email protected]> * context.tf updated to v0.24.1, minimum required Terraform version bumped to 0.13.0 when needed, readme updated (cloudposse#33) Co-authored-by: Jürgen W <[email protected]> Co-authored-by: actions-bot <[email protected]> Co-authored-by: Nuru <[email protected]> Co-authored-by: Cloud Posse Bot (CI/CD) <[email protected]> Co-authored-by: Maxim Mironenko <[email protected]>
what
why