Skip to content

Commit d83cbd7

Browse files
authored
[Docs Site] Fix unmatched CODEOWNERS entries (#19446)
1 parent 94d19f9 commit d83cbd7

File tree

4 files changed

+111
-9
lines changed

4 files changed

+111
-9
lines changed

.github/CODEOWNERS

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
/.github/actions/assign-pr/index.js @cloudflare/pcx-technical-writing
1111
/.github/styles/cloudflare/spelling-exceptions.txt @cloudflare/pcx-technical-writing
1212
/src/components/ @cloudflare/developer-advocacy @kristianfreeman @kodster28 @pedrosousa @marciocloudflare @haleycode @maxvp @GregBrimble @KianNH @WalshyDev
13-
/functions/ @cloudflare/developer-advocacy @kristianfreeman @kodster28 @pedrosousa @haleycode @marciocloudflare @maxvp @GregBrimble @KianNH @WalshyDev
1413
*.js @cloudflare/developer-advocacy @kristianfreeman @kodster28 @pedrosousa @haleycode @maxvp @marciocloudflare @GregBrimble @KianNH @WalshyDev
1514
*.ts @cloudflare/developer-advocacy @kristianfreeman @kodster28 @pedrosousa @haleycode @maxvp @marciocloudflare @GregBrimble @KianNH @WalshyDev
1615
/src/content/workers-ai-models/ @craigsdennis @pedrosousa @cloudflare/pcx-technical-writing
@@ -20,7 +19,6 @@
2019

2120
/src/content/docs/ai-gateway/ @kathayl @G4brym @mchenco @daisyfaithauma @cloudflare/pcx-technical-writing
2221
/src/content/docs/workers-ai/ @rita3ko @craigsdennis @markdembo @mchenco @daisyfaithauma @cloudflare/pcx-technical-writing
23-
/src/content/docs/workers-ai/static @mchenco @craigsdennis @cloudflare/pcx-technical-writing
2422
/src/content/docs/vectorize/ @elithrar @vy-ton @sejoker @mchenco @cloudflare/pcx-technical-writing
2523
/src/content/partials/vectorize/ @elithrar @mchenco @sejoker @cloudflare/pcx-technical-writing
2624
/src/content/changelogs/workers-ai.yaml @kathayl @G4brym @mchenco @daisyfaithauma @cloudflare/pcx-technical-writing
@@ -56,7 +54,7 @@
5654
/src/content/docs/cloudflare-one/policies/access/ @kennyj42 @ranbel @cloudflare/pcx-technical-writing
5755
/src/content/docs/cloudflare-one/connections/connect-devices/ @kkrum @kokolocomotion1 @ranbel @cloudflare/pcx-technical-writing
5856
/src/content/docs/cloudflare-one/connections/connect-networks/ @ranbel @cloudflare/pcx-technical-writing
59-
/src/content/docs/cloudflare-one/applications/scan-apps/ @maxvp @cloudflare/pcx-technical-writing
57+
/src/content/docs/cloudflare-one/applications/casb/ @maxvp @cloudflare/pcx-technical-writing
6058
/src/content/docs/cloudflare-one/policies/gateway/ @maxvp @cloudflare/pcx-technical-writing
6159
/src/content/docs/cloudflare-one/policies/browser-isolation/ @maxvp @ranbel @cloudflare/pcx-technical-writing
6260
/src/content/docs/cloudflare-one/policies/data-loss-prevention/ @maxvp @cloudflare/pcx-technical-writing
@@ -78,7 +76,6 @@
7876
/src/content/docs/cloudflare-for-platforms/cloudflare-for-saas/ @RebeccaTamachiro @cloudflare/pcx-technical-writing
7977
/src/content/docs/dns/ @RebeccaTamachiro @cloudflare/pcx-technical-writing
8078
/src/content/docs/fundamentals/ @dcpena @cloudflare/pcx-technical-writing
81-
/src/content/docs/fundamentals/account-and-billing/ @dcpena @cloudflare/pcx-technical-writing
8279
/src/content/docs/learning-paths/ @cloudflare/pcx-technical-writing
8380
/src/content/docs/network-error-logging/ @dcpena @cloudflare/pcx-technical-writing
8481
/src/content/docs/network-interconnect/ @marciocloudflare @cloudflare/pcx-technical-writing
@@ -126,17 +123,17 @@
126123
/src/content/docs/workers/runtime-apis/ @irvinebroque @jasnell @mikenomitch @GregBrimble @ToriLindsay @cloudflare/pcx-technical-writing
127124
/src/content/docs/workers/runtime-apis/bindings/ @irvinebroque @mikenomitch @GregBrimble @ToriLindsay @cloudflare/pcx-technical-writing
128125
/src/content/docs/workers/platform/ @irvinebroque @GregBrimble @cloudflare/deploy-config @cloudflare/pcx-technical-writing
129-
/src/content/docs/workers/configuration/compatibility-dates/ @irvinebroque @mikenomitch @GregBrimble @ToriLindsay @cloudflare/deploy-config @cloudflare/pcx-technical-writing
130-
/src/content/docs/workers/configuration/compatibility-flags/ @irvinebroque @mikenomitch @GregBrimble @ToriLindsay @cloudflare/deploy-config @cloudflare/pcx-technical-writing
126+
/src/content/docs/workers/configuration/compatibility-dates.mdx @irvinebroque @mikenomitch @GregBrimble @ToriLindsay @cloudflare/deploy-config @cloudflare/pcx-technical-writing
127+
/src/content/docs/workers/configuration/compatibility-flags.mdx @irvinebroque @mikenomitch @GregBrimble @ToriLindsay @cloudflare/deploy-config @cloudflare/pcx-technical-writing
131128

132-
/src/content/docs/workers/reference/migrate-to-module-workers/ @irvinebroque @GregBrimble @ToriLindsay @cloudflare/deploy-config @cloudflare/pcx-technical-writing
133-
/src/content/docs/workers/reference/security-model/ @irvinebroque @GregBrimble @ToriLindsay @cloudflare/pcx-technical-writing
129+
/src/content/docs/workers/reference/migrate-to-module-workers.mdx @irvinebroque @GregBrimble @ToriLindsay @cloudflare/deploy-config @cloudflare/pcx-technical-writing
130+
/src/content/docs/workers/reference/security-model.mdx @irvinebroque @GregBrimble @ToriLindsay @cloudflare/pcx-technical-writing
134131
/src/content/compatibility-flags/ @irvinebroque @mikenomitch @GregBrimble @cloudflare/pcx-technical-writing
135132
/src/content/docs/workers/wrangler/ @cloudflare/wrangler @cloudflare/wrangler-friends @irvinebroque @ToriLindsay @cloudflare/pcx-technical-writing
136133
/src/content/docs/workers/frameworks/ @igorminar @dario-piotrowicz @jculvey @aninibread @GregBrimble @ToriLindsay @cloudflare/pcx-technical-writing
137134
/src/content/docs/pages/framework-guides/ @igorminar @dario-piotrowicz @jculvey @aninibread @GregBrimble @ToriLindsay @cloudflare/pcx-technical-writing
138135
/src/content/docs/analytics/analytics-engine/ @irvinebroque @elithrar @cloudflare/pcx-technical-writing
139-
/cloudflare-for-platforms/workers-for-platforms/ @irvinebroque @angelampcosta @GregBrimble @cloudflare/deploy-config @cloudflare/pcx-technical-writing
136+
/src/content/docs/cloudflare-for-platforms/workers-for-platforms/ @irvinebroque @angelampcosta @GregBrimble @cloudflare/deploy-config @cloudflare/pcx-technical-writing
140137
/src/content/docs/workers/observability/ @irvinebroque @mikenomitch @rohinlohe @ToriLindsay @cloudflare/pcx-technical-writing
141138
/src/content/docs/workers/static-assets @irvinebroque @GregBrimble @WalshyDev @ToriLindsay @cloudflare/deploy-config @cloudflare/pcx-technical-writing
142139
/src/content/docs/workflows/ @ToriLindsay @elithrar @celso @sidharthachatterjee @cloudflare/pcx-technical-writing

bin/validate-codeowners.ts

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
import { spawnSync } from "node:child_process";
2+
import { loadOwners, matchPattern } from "codeowners-utils";
3+
4+
let owners = await loadOwners(process.cwd());
5+
6+
if (!owners) {
7+
throw new Error("Unable to load CODEOWNERS file.");
8+
}
9+
10+
const result = spawnSync("git", ["ls-files"], { encoding: "utf-8" });
11+
const files = result.stdout.trim().split("\n");
12+
13+
owners = owners.filter((entry) => entry.pattern !== "*");
14+
15+
const unmatchedPatterns = owners
16+
.filter((entry) => {
17+
for (const file of files) {
18+
const match = matchPattern(file, entry.pattern);
19+
20+
if (match) {
21+
return false;
22+
}
23+
}
24+
25+
return true;
26+
})
27+
.map((entry) => entry.pattern);
28+
29+
if (unmatchedPatterns.length > 0) {
30+
console.error(
31+
`CODEOWNERS contains ${unmatchedPatterns.length} patterns that do not match any files in the repository:`,
32+
);
33+
console.error(unmatchedPatterns);
34+
}

package-lock.json

Lines changed: 70 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@
5252
"astro-breadcrumbs": "3.3.1",
5353
"astro-icon": "1.1.5",
5454
"astro-live-code": "0.0.5",
55+
"codeowners-utils": "1.0.2",
5556
"date-fns": "4.1.0",
5657
"dedent": "1.5.3",
5758
"dompurify": "3.2.3",

0 commit comments

Comments
 (0)