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

GH Actions shortlog: check out head ref #6471

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Conversation

MetRonnie
Copy link
Member

This avoids the action failing on PRs due to the merge commit checked out by default having different author details to the locally pushed changes by that contributor

@MetRonnie MetRonnie added small infrastructure GH Actions, Codecov etc. labels Nov 11, 2024
@MetRonnie MetRonnie self-assigned this Nov 11, 2024
This avoids the action failing on PRs due to the merge commit checked out by default having different author details to the locally pushed changes by that contributor
@MetRonnie
Copy link
Member Author

MetRonnie commented Nov 11, 2024

Observe that the checkout commit matches the pushed commit: https://github.com/cylc/cylc-flow/actions/runs/11777691847/job/32802589334?pr=6471#step:2:362

Tested on workflow_dispatch where the behaviour is unchanged: https://github.com/MetRonnie/cylc-flow/actions/runs/11777695976/job/32802597946

@oliver-sanders oliver-sanders merged commit c4ec05b into cylc:8.3.x Nov 11, 2024
12 checks passed
@MetRonnie MetRonnie deleted the actions branch November 11, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure GH Actions, Codecov etc. small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants