From 8e329139ceac045fc7c547dff9ffb60cabd45047 Mon Sep 17 00:00:00 2001 From: Fabio Pitino <fpitino@gitlab.com> Date: Fri, 9 Feb 2024 22:08:06 +0000 Subject: [PATCH] Fix CODEOWNERS rule to ignore Verify specs for lib The generic rule `/**/lib/**/ci/` was not intended to capture spec files. It turns out that was catching directories like `/lib/spec/**/ci/` as well as `/ee/lib/spec/**/ci/`. Changelog: fixed --- .gitlab/CODEOWNERS | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/.gitlab/CODEOWNERS b/.gitlab/CODEOWNERS index 4e3c7bfb515e..4ab4e0f406b8 100644 --- a/.gitlab/CODEOWNERS +++ b/.gitlab/CODEOWNERS @@ -1237,9 +1237,11 @@ lib/gitlab/checks/** # opportunity to refine specific rules defined in this section. # Note that frontend, CI templates and other concerns should be kept within # the same section. -/**/app/**/ci/ -/**/lib/**/ci/ -/**/app/**/merge_trains/ +/app/**/ci/ +/ee/app/**/ci/ +/lib/**/ci/ +/ee/lib/**/ci/ +/ee/app/**/merge_trains/ /app/controllers/admin/jobs_controller.rb /app/controllers/admin/runner_projects_controller.rb /app/controllers/admin/runners_controller.rb -- GitLab