Skip to content

Do not call "git diff" when listing PRs#33817

Merged
wxiaoguang merged 3 commits into
go-gitea:release/v1.23from
wxiaoguang:fix-pr-list-diff
Mar 8, 2025
Merged

Do not call "git diff" when listing PRs#33817
wxiaoguang merged 3 commits into
go-gitea:release/v1.23from
wxiaoguang:fix-pr-list-diff

Conversation

@wxiaoguang
Copy link
Copy Markdown
Contributor

Fix #31492

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 7, 2025
@GiteaBot GiteaBot added this to the 1.23.6 milestone Mar 7, 2025
@pull-request-size pull-request-size Bot added size/M and removed size/S labels Mar 7, 2025
@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 7, 2025
@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 Mar 8, 2025
@wxiaoguang wxiaoguang enabled auto-merge (squash) March 8, 2025 07:19
@wxiaoguang
Copy link
Copy Markdown
Contributor Author

wxiaoguang commented Mar 8, 2025

The one for main branch is still pending. Decouple diff stats query from actual diffing #33810

@wxiaoguang wxiaoguang merged commit 869ee4f into go-gitea:release/v1.23 Mar 8, 2025
@wxiaoguang wxiaoguang deleted the fix-pr-list-diff branch March 8, 2025 09:43
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Jun 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants