From 2181b6f7d43fa17cc1b686077d48843d2bd25ab3 Mon Sep 17 00:00:00 2001 From: yakew7_ Date: Fri, 25 Sep 2026 17:10:46 +0530 Subject: [PATCH] chore: add /assets/ catch-all to CODEOWNERS so assets/demo.gif has an owner #711 added assets/demo.gif with no matching CODEOWNERS entry, which fails test_every_tracked_file_has_at_least_one_codeowners_owner. The test only runs in the profiler job, which skips docs-only changes, so it went unnoticed. A /assets/ catch-all placed above the specific asset entries covers this and any future README media without changing existing owners. --- .github/CODEOWNERS | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 74d21284..af56f2ee 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -55,6 +55,10 @@ # ========================= # Website & web profiler # ========================= +# Catch-all for anything under assets/ not matched by a more specific entry +# below (e.g. README media like assets/demo.gif). Must stay above the +# specific /assets/ entries - CODEOWNERS uses the last matching pattern. +/assets/ @yakew7 /assets/profiler-engine.js @yakew7 @ahmdkaml /assets/profiler-compare.js @yakew7 @ahmdkaml /assets/profiler-ui.js @yakew7 @ahmdkaml