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
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