Merge pull request #159091 from rickystewart/blathers/backport-release-24.3-159081

rickystewart · web-flow · commit 27f0afc34b7c · 2025-12-09T17:49:24.000-06:00
release-24.3: codeowners: include gitignore'd files in test file lookup
diff --git a/pkg/internal/codeowners/codeowners.go b/pkg/internal/codeowners/codeowners.go
@@ -208,7 +208,7 @@ func getFile(packageName, testName string) (res string, logs []string, err error
 			return
 		}
 		cmd := exec.Command(`/bin/bash`, `-c`,
-			fmt.Sprintf(`cd "$(git rev-parse --show-toplevel)" && git grep -n 'func %s(' '%s/*_test.go'`,
+			fmt.Sprintf(`cd "$(git rev-parse --show-toplevel)" && git grep --no-index -n 'func %s(' '%s/*_test.go'`,
 				testName, packageName))
 		// This command returns output such as:
 		// ../ccl/storageccl/export_test.go:31:func TestExportCmd(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,7 @@ func getFile(packageName, testName string) (res string, logs []string, err error`
`208`	`208`	`return`
`209`	`209`	`}`
`210`	`210`	cmd := exec.Command(`/bin/bash`, `-c`,
`211`		- fmt.Sprintf(`cd "$(git rev-parse --show-toplevel)" && git grep -n 'func %s(' '%s/*_test.go'`,
	`211`	+ fmt.Sprintf(`cd "$(git rev-parse --show-toplevel)" && git grep --no-index -n 'func %s(' '%s/*_test.go'`,
`212`	`212`	`testName, packageName))`
`213`	`213`	`// This command returns output such as:`
`214`	`214`	`// ../ccl/storageccl/export_test.go:31:func TestExportCmd(t *testing.T) {`