Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix required contexts and commit status matching bug#34815

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
lunny merged 4 commits intogo-gitea:mainfromZettat123:bugfix/issue-34504
Jun 22, 2025

Conversation

@Zettat123
Copy link
Contributor

Fix#34504

Since one required context can match more than one commit statuses, we should not directly compare the lengths ofrequiredCommitStatuses andrequiredContexts

@GiteaBotGiteaBot added the lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelJun 22, 2025
@github-actionsgithub-actionsbot added the modifies/goPull requests that update Go code labelJun 22, 2025
@GiteaBotGiteaBot added lgtm/need 1This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelsJun 22, 2025
@lunnylunny added type/bug backport/v1.24This PR should be backported to Gitea 1.24 labelsJun 22, 2025
@GiteaBotGiteaBot added lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1This PR needs approval from one additional maintainer to be merged. labelsJun 22, 2025
wxiaoguangand others added2 commitsJune 22, 2025 21:39
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>Signed-off-by: Zettat123 <zettat123@gmail.com>
@lunnylunny added the reviewed/wait-mergeThis pull request is part of the merge queue. It will be merged soon. labelJun 22, 2025
@lunnylunnyenabled auto-merge (squash)June 22, 2025 23:19
@lunnylunny merged commit618e2d8 intogo-gitea:mainJun 22, 2025
26 checks passed
@GiteaBotGiteaBot added this to the1.25.0 milestoneJun 22, 2025
@GiteaBot
Copy link
Collaborator

I was unable to create a backport for 1.24.@Zettat123, please send one manually. 🍵

go run ./contrib/backport 34815...  // fix git conflicts if anygo run ./contrib/backport --continue

@GiteaBotGiteaBot added backport/manualNo power to the bots! Create your backport yourself! and removed reviewed/wait-mergeThis pull request is part of the merge queue. It will be merged soon. labelsJun 22, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull requestJun 23, 2025
* giteaofficial/main: (22 commits)  [skip ci] Updated translations via Crowdin  Fix required contexts and commit status matching bug (go-gitea#34815)  Add "Cancel workflow run" button to Actions list page (go-gitea#34817)  Use `shallowRef` instead of `ref` in `.vue` files where possible (go-gitea#34813)  Edit file workflow for creating a fork and proposing changes (go-gitea#34240)  Refactor packages (go-gitea#34777)  Refactor wiki (go-gitea#34805)  [skip ci] Updated translations via Crowdin  Upgrade htmx to 2.0.5 (go-gitea#34809)  Forks repository list page follow other repositories page (go-gitea#34784)  Add post-installation redirect based on admin account status (go-gitea#34493)  Rework delete org and rename org UI (go-gitea#34762)  Refactor editor (go-gitea#34780)  Improve img lazy loading (go-gitea#34804)  [skip ci] Updated translations via Crowdin  Fix Feishu webhook signature verification (go-gitea#34788)  Upgrade chi to v5.2.2 (go-gitea#34798)  Fix the issue of abnormal interface when there is no issue-item on the project page (go-gitea#34791)  Fix container range bug (go-gitea#34795)  Fix OCI manifest parser (go-gitea#34797)  ...
@Zettat123
Copy link
ContributorAuthor

I was unable to create a backport for 1.24.@Zettat123, please send one manually. 🍵

go run ./contrib/backport 34815...  // fix git conflicts if anygo run ./contrib/backport --continue

-->#34829

@lunnylunny added the backport/doneAll backports for this PR have been created labelJun 23, 2025
wxiaoguang pushed a commit that referenced this pull requestJun 23, 2025
Backport#34815Fix#34504Since one required context can match more than one commit statuses, weshould not directly compare the lengths of `requiredCommitStatuses` and`requiredContexts`
@go-giteago-gitea locked asresolvedand limited conversation to collaboratorsSep 24, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@lunnylunnylunny approved these changes

@wxiaoguangwxiaoguangwxiaoguang approved these changes

Assignees

No one assigned

Labels

backport/doneAll backports for this PR have been createdbackport/manualNo power to the bots! Create your backport yourself!backport/v1.24This PR should be backported to Gitea 1.24lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore.modifies/goPull requests that update Go codetype/bug

Projects

None yet

Milestone

1.25.0

Development

Successfully merging this pull request may close these issues.

Branch protection with required status check

4 participants

@Zettat123@GiteaBot@lunny@wxiaoguang

[8]ページ先頭

©2009-2025 Movatter.jp