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 autoupdate for pulumi/pulumi #632
Conversation
Diff for pulumi-azuread with merge commit 80cb9fb |
Diff for pulumi-random with merge commit 80cb9fb |
Diff for pulumi-gcp with merge commit 80cb9fb |
Diff for pulumi-azure with merge commit 80cb9fb |
Diff for pulumi-aws with merge commit 80cb9fb |
@@ -35,22 +35,33 @@ jobs: | |||
id: gomod | |||
run: >- |
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.
This PR is fine, but the other way of fixing this is to use the |
character instead. |
makes more sense for lines of code. >
is designed for allowing wrapping for blocks of text - hence you have to use 2 line breaks to get 1 real line break output.
run: >- | |
run: | |
Reference: https://yaml-multiline.info/
git commit -m "Updated modules" | ||
|
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.
This is a cool workflow, TIL about repo-sync/pull-request@v2.6.2 automation piece! Nice!
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 but +1 to what Daniel said.
I will merge this and fix the yaml multiline subsequently. This was blindly copied from what we do in the native providers. |
Sample run here: https://github.com/pulumi/pulumi-terraform-bridge/actions/runs/3388759763/jobs/5631104118