From 98e435490df32c9c19e882891b12680c04d8fe61 Mon Sep 17 00:00:00 2001 From: Deluan Date: Sat, 5 Sep 2026 13:59:47 -0400 Subject: [PATCH] ci: treat everything under tests/ as a Go change tests/ holds only Go test scaffolding: the shared mocks, tests/fixtures, and tests/navidrome-test.toml, which tests.Init reads for every suite. A change to any of it can alter a Go test result, and the directory can never collide with a frontend or docs path. This deliberately does not cover testdata/ generally. Measured over the full history, 99 of 103 pull requests touching testdata/ or tests/fixtures/ already matched the Go filter, and the single missed pull request was a frontend one that a broader pattern would have made worse by running the whole Go matrix. --- .github/workflows/detect-changes.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/detect-changes.sh b/.github/workflows/detect-changes.sh index d1a6760ad..3146abf66 100755 --- a/.github/workflows/detect-changes.sh +++ b/.github/workflows/detect-changes.sh @@ -20,7 +20,7 @@ set -uo pipefail export LC_ALL=C -GO_RE='(\.go$|(^|/)go\.(mod|sum)$|^Makefile$|^\.golangci\.yml$|^resources/|^db/migrations/)' +GO_RE='(\.go$|(^|/)go\.(mod|sum)$|^Makefile$|^\.golangci\.yml$|^resources/|^db/migrations/|^tests/)' JS_RE='^ui/' I18N_RE='(^resources/i18n/|^ui/src/i18n/en\.json$|^\.github/workflows/validate-translations\.sh$)' DOC_ONLY_RE='(\.md$|^LICENSE$|^\.git-blame-ignore-revs$|^\.gitignore$|^\.devcontainer/)'