Skip to content

Conversation

wxiaoguang
Copy link
Contributor

@wxiaoguang wxiaoguang commented Jul 17, 2025

Follow up #35096, fix #35095, fix #35115 and add more tests

The old code used some fragile behaviors which depend on the "nil" receiver. This PR should be a complete fix for more edge cases.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 17, 2025
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Jul 17, 2025
@wxiaoguang wxiaoguang added type/bug backport/v1.24 This PR should be backported to Gitea 1.24 labels Jul 17, 2025
@wxiaoguang wxiaoguang added this to the 1.25.0 milestone Jul 17, 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 Jul 17, 2025
@wxiaoguang wxiaoguang added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Jul 17, 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 Jul 18, 2025
@wxiaoguang wxiaoguang enabled auto-merge (squash) July 18, 2025 08:57
@wxiaoguang wxiaoguang merged commit 8f91bfe into go-gitea:main Jul 18, 2025
26 checks passed
@wxiaoguang wxiaoguang deleted the fix-submodule-nil branch July 18, 2025 09:54
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Jul 18, 2025
Follow up go-gitea#35096, fix go-gitea#35095, fix go-gitea#35115 and add more tests

The old code used some fragile behaviors which depend on the "nil"
receiver. This PR should be a complete fix for more edge cases.
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Jul 18, 2025
wxiaoguang added a commit that referenced this pull request Jul 18, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jul 22, 2025
* giteaofficial/main:
  [skip ci] Updated translations via Crowdin
  Fix session gob (go-gitea#35128)
  Replace `setup-python` with `setup-uv` (go-gitea#35116)
  Don't use full-file highlight when there is a git diff textconv (go-gitea#35114)
  Fix submodule parsing when the gitmodules is missing (go-gitea#35109)
  Align `issue-title-buttons` with `list-header` (go-gitea#35018)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/done All backports for this PR have been created backport/v1.24 This PR should be backported to Gitea 1.24 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/go Pull requests that update Go code skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Projects
None yet
4 participants