Skip to content

pull: Fix CODEOWNERS absolute path matching. (#37244)#37264

Merged
silverwind merged 1 commit intogo-gitea:release/v1.26from
GiteaBot:backport-37244-v1.26
Apr 17, 2026
Merged

pull: Fix CODEOWNERS absolute path matching. (#37244)#37264
silverwind merged 1 commit intogo-gitea:release/v1.26from
GiteaBot:backport-37244-v1.26

Conversation

@GiteaBot
Copy link
Copy Markdown
Collaborator

Backport #37244 by @JoeGruffins

Patterns starting with "/" (e.g. /docs/.*.md) never matched because git returns relative paths without a leading slash. Strip the leading "/" before compiling the regex since the ^...$ anchoring already provides root-relative semantics.

closes #28107

Patterns starting with "/" (e.g. /docs/.*\.md) never matched because git
returns relative paths without a leading slash. Strip the leading "/"
before compiling the regex since the ^...$ anchoring already provides
root-relative semantics.

Fixes: go-gitea#28107
Co-authored-by: silverwind <me@silverwind.io>
Co-authored-by: Giteabot <teabot@gitea.io>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 17, 2026
@GiteaBot GiteaBot added this to the 1.26.0 milestone Apr 17, 2026
@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 Apr 17, 2026
@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 Apr 17, 2026
@silverwind silverwind enabled auto-merge (squash) April 17, 2026 22:40
@silverwind silverwind merged commit 26a618a into go-gitea:release/v1.26 Apr 17, 2026
26 checks passed
@lunny lunny added the type/bug label Apr 17, 2026
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. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants