Skip to content

fix(actions): wrong assumption that run id always >= job id (#37737)#37742

Merged
silverwind merged 1 commit into
go-gitea:release/v1.26from
Zettat123:backport/37737-to-v1.26
May 17, 2026
Merged

fix(actions): wrong assumption that run id always >= job id (#37737)#37742
silverwind merged 1 commit into
go-gitea:release/v1.26from
Zettat123:backport/37737-to-v1.26

Conversation

@Zettat123
Copy link
Copy Markdown
Contributor

Backport #37737

Fix #37734

Follow up #37008

The jobNum >= runNum check is useless. Removed it to support job_id < run_id

…#37737)

Fix go-gitea#37734

Follow up go-gitea#37008

The `jobNum >= runNum` check is useless. Removed it to support `job_id <
run_id`
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 17, 2026
@GiteaBot GiteaBot added this to the 1.26.2 milestone May 17, 2026
@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 May 17, 2026
@lunny lunny added the type/bug label May 17, 2026
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels May 17, 2026
@silverwind silverwind merged commit 1d7b849 into go-gitea:release/v1.26 May 17, 2026
26 checks passed
@Zettat123 Zettat123 deleted the backport/37737-to-v1.26 branch May 17, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants