Skip to content

Commit 825ab10

Browse files
committed
fix
1 parent 2916a4c commit 825ab10

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ require (
3737
github.com/charmbracelet/git-lfs-transfer v0.1.1-0.20251013092601-6327009efd21
3838
github.com/chi-middleware/proxy v1.1.1
3939
github.com/dimiro1/reply v0.0.0-20200315094148-d0136a4c9e21
40+
github.com/dlclark/regexp2 v1.11.5
4041
github.com/dsnet/compress v0.0.2-0.20230904184137-39efe44ab707
4142
github.com/dustin/go-humanize v1.0.1
4243
github.com/editorconfig/editorconfig-core-go/v2 v2.6.4
@@ -183,7 +184,6 @@ require (
183184
github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc // indirect
184185
github.com/davidmz/go-pageant v1.0.2 // indirect
185186
github.com/dgryski/go-rendezvous v0.0.0-20200823014737-9f7001d12a5f // indirect
186-
github.com/dlclark/regexp2 v1.11.5 // indirect
187187
github.com/emersion/go-sasl v0.0.0-20241020182733-b788ff22d5a6 // indirect
188188
github.com/fatih/color v1.18.0 // indirect
189189
github.com/fxamacker/cbor/v2 v2.9.0 // indirect

services/issue/pull.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,9 @@ func PullRequestCodeOwnersReview(ctx context.Context, pr *issues_model.PullReque
9595
uniqTeams := make(map[string]*org_model.Team)
9696
for _, rule := range rules {
9797
for _, f := range changedFiles {
98-
if (rule.Rule.MatchString(f) && !rule.Negative) || (!rule.Rule.MatchString(f) && rule.Negative) {
98+
shouldMatch := !rule.Negative
99+
matched, _ := rule.Rule.MatchString(f)
100+
if matched == shouldMatch {
99101
for _, u := range rule.Users {
100102
uniqUsers[u.ID] = u
101103
}

0 commit comments

Comments
 (0)