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.

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

workflows: ensure auto-pr always runs on the right commit #132

Merged
merged 3 commits into from
Nov 19, 2024
Merged

Conversation

woodruffw
Copy link
Member

This fixes a piece of lagged state between audit.yml and auto-pr.yml: now that auto-pr.yml is a reusable workflow, it runs on the GITHUB_REF that audit.yml is triggered on by default.

This means that it misses the most recent git push containing updated vulnerability information, which in turn meant that the intended PR step always ran one cycle behind the audit step.

This PR fixes the behavior by adding a ref: input that feeds into actions/checkout, ensuring that we check out the just-pushed change.

@woodruffw woodruffw self-assigned this Nov 19, 2024
Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
@woodruffw woodruffw merged commit 55de727 into main Nov 19, 2024
3 checks passed
@woodruffw woodruffw deleted the ww/fix-ref branch November 19, 2024 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant