Skip to content

Comments

BUG: GHA: Cancel-in-progress event if not the same head SHA#5042

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
SimonRit:cancel-in-progress
Dec 11, 2024
Merged

BUG: GHA: Cancel-in-progress event if not the same head SHA#5042
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
SimonRit:cancel-in-progress

Conversation

@SimonRit
Copy link

Grouping based on github.event.pull_request.head.sha, i.e. "the commit ID for the last commit to the head branch of the pull request" only cancels commits with the same SHA. With this PR, on-going workflows of the same PR would be cancelled.

See #5019 (comment).

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

@github-actions github-actions bot added type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots labels Dec 11, 2024
Grouping based on github.event.pull_request.head.sha, i.e. "the commit
ID for the last commit to the head branch of the pull request" only
cancels commits with the same SHA. With this PR, on-going workflows of
the same PR would be cancelled.
@SimonRit SimonRit changed the title BUG: GHA: Do not cancel-in-progress same base SHA BUG: GHA: Cancel-in-progress event if not the same head SHA Dec 11, 2024
@dzenanz dzenanz requested a review from blowekamp December 11, 2024 16:08
@blowekamp
Copy link
Member

Can we test this with a force push/update on this branch?

@dzenanz
Copy link
Member

dzenanz commented Dec 11, 2024

Simon already made a force-push, which seems to cause cancellations: https://github.com/InsightSoftwareConsortium/ITK/actions/runs/12280142773/job/34265846027

Copy link
Member

@blowekamp blowekamp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hjmjohnson hjmjohnson merged commit e27f2cf into InsightSoftwareConsortium:master Dec 11, 2024
@SimonRit SimonRit deleted the cancel-in-progress branch January 27, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants