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

Fix auto concurrency cancellation skips commit status updates #33764

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ChristopherHX
Copy link
Contributor

@ChristopherHX ChristopherHX commented Mar 1, 2025

  • add missing commit status
  • conflicts with concurrency support

Closes #33763


Since this conflics with #32751 that is planned for Gitea v1.24, this might make more sense as a backport to 1.23 if considered. Had this change laying around, because I'm not on a branch with "auto concurrency" replaced by "concurrency".

A 1.23 backport that resolves the conflict is here https://github.com/go-gitea/gitea/compare/release/v1.23...ChristopherHX:gitea:fix-actions-commit-status-update-1-23?expand=1

* add missing commit status
* conflicts with concurrency support
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 1, 2025
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Mar 1, 2025
@github-actions github-actions bot added modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code labels Mar 1, 2025
@lunny lunny requested a review from Zettat123 March 2, 2025 02:28
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 2, 2025
@lunny lunny added type/bug backport/v1.23 This PR should be backported to Gitea 1.23 labels Mar 2, 2025
@lunny lunny mentioned this pull request Mar 3, 2025
lunny added a commit that referenced this pull request Mar 4, 2025
Wait tomorrow's Golang version.
Maybe wait backport of #33764 and #33744, #33789

---------

Co-authored-by: metiftikci <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/v1.23 This PR should be backported to Gitea 1.23 lgtm/need 1 This PR needs approval from one additional maintainer to be merged. modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code size/M Denotes a PR that changes 30-99 lines, ignoring generated files. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto Concurrency CancelPreviousJobs does not update the commit status to cancelled
3 participants