Skip to content

Conversation

@dmach
Copy link
Contributor

@dmach dmach commented Nov 12, 2025

GITEA_PR_ID is already part of the env variables available in the githooks,
but it contains a database ID instead of commonly used index that is part of owner/repo!index

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 12, 2025
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Nov 12, 2025
@dmach dmach force-pushed the githooks-add-pr-index branch from 758c3a0 to a2b79d9 Compare November 12, 2025 16:32
@dmach dmach force-pushed the githooks-add-pr-index branch from a2b79d9 to b2e09fe Compare November 13, 2025 07:30
@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 Nov 13, 2025
@wxiaoguang wxiaoguang added the type/enhancement An improvement of existing functionality label Nov 13, 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 Nov 13, 2025
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 14, 2025
@lunny lunny added this to the 1.26.0 milestone Nov 14, 2025
@lunny lunny enabled auto-merge (squash) November 14, 2025 02:55
@wxiaoguang wxiaoguang disabled auto-merge November 14, 2025 03:06
@wxiaoguang wxiaoguang enabled auto-merge (squash) November 14, 2025 03:53
@wxiaoguang wxiaoguang merged commit d6dc531 into go-gitea:main Nov 14, 2025
25 checks passed
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 14, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 17, 2025
* giteaofficial/main:
  Fix diff blob excerpt expansion (go-gitea#35922)
  Add GITEA_PR_INDEX env variable to githooks (go-gitea#35938)
  Fix container push tag overwriting (go-gitea#35936)
  Upgrade deps golang.org/x/crypto (go-gitea#35952)
  Fix corrupted external render content (go-gitea#35946)
  Don't show unnecessary error message to end users for DeleteBranchAfterMerge (go-gitea#35937)
  Limit reading bytes instead of ReadAll (go-gitea#35928)
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. modifies/go Pull requests that update Go code type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants