From b9fce08e93d6ce426e5c216c2c794d0b77b6e183 Mon Sep 17 00:00:00 2001 From: Waldemar Zurowski Date: Tue, 30 Apr 2024 16:48:34 +0100 Subject: [PATCH 1/2] ES-1978: corrected syntax in CODEOWNERS file * `.ci/*` only catches files directly in `.ci` folder, any subfolders are not affected * `/.ci/` correctly sets ownership of all files in that folder to Infrastructure and Release team * same for `.gradle` folder --- .github/CODEOWNERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index d54c5b4..d123734 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -3,5 +3,5 @@ *.gradle @corda/infrastructure-release gradle.properties @corda/corda5-team-leads Jenkinsfile @corda/infrastructure-release -.ci/* @corda/infrastructure-release -gradle/* @corda/infrastructure-release +/.ci/ @corda/infrastructure-release +/gradle/ @corda/infrastructure-release From 2bae2578a40468509d4039a728f674d78451832b Mon Sep 17 00:00:00 2001 From: Waldemar Zurowski Date: Tue, 30 Apr 2024 16:52:19 +0100 Subject: [PATCH 2/2] Fixes --- .github/CODEOWNERS | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index d123734..2f00ac8 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,7 +1,9 @@ * @corda/rest -# Build scripts should be audited by BLT -*.gradle @corda/infrastructure-release -gradle.properties @corda/corda5-team-leads -Jenkinsfile @corda/infrastructure-release -/.ci/ @corda/infrastructure-release -/gradle/ @corda/infrastructure-release +# Build scripts should be audited by Infrastructure & Release team +*.gradle @corda/infrastructure-release +gradle.properties @corda/corda5-team-leads +Jenkinsfile @corda/infrastructure-release +/.ci/ @corda/infrastructure-release +/.github/ @corda/infrastructure-release +/.github/CODEOWNERS @corda/rest # keep CODEOWNERS owned by the Rest team +/gradle/ @corda/infrastructure-release