From 9ff8284c87fe0db37e56da7f5e928f4da8c90bc3 Mon Sep 17 00:00:00 2001 From: Michal Checinski Date: Fri, 7 Jun 2024 16:47:42 +0200 Subject: [PATCH 1/3] BRE-101: Remove dept-devops from CODEOWNERS --- .github/CODEOWNERS | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index a067f879cc82..358684613e21 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -4,14 +4,6 @@ # # https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners -# DevOps for Actions and other workflow changes -.github/workflows @bitwarden/dept-devops - -# DevOps for Docker changes -**/Dockerfile @bitwarden/dept-devops -**/*.Dockerfile @bitwarden/dept-devops -**/.dockerignore @bitwarden/dept-devops - # Database Operations for database changes src/Sql/** @bitwarden/dept-dbops util/EfShared/** @bitwarden/dept-dbops @@ -55,6 +47,9 @@ bitwarden_license/src/test/Scim.ScimTest @bitwarden/team-admin-console-dev src/Admin/Controllers/ToolsController.cs @bitwarden/team-billing-dev src/Admin/Views/Tools @bitwarden/team-billing-dev -# Multiple owners - DO NOT REMOVE (DevOps) +# Multiple owners - DO NOT REMOVE (BRE) **/packages.lock.json Directory.Build.props + +# BRE for release workflow changes +.github/workflows/release.yml @bitwarden/dept-bre From 568a67c7bc043e70824b82b936f75870bbea7548 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Ch=C4=99ci=C5=84ski?= Date: Mon, 14 Oct 2024 12:54:49 +0200 Subject: [PATCH 2/3] Update codeowners according to BRE-101 Update CODEOWNERS for CI/CD Partnership Phase 2 --- .github/CODEOWNERS | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 358684613e21..513ec7c42c43 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -4,6 +4,21 @@ # # https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners +## Docker files have shared ownership ## +**/Dockerfile +**/*.Dockerfile +**/.dockerignore +**/entrypoint.sh + +## BRE team owns these workflows ## +.github/workflows/publish.yml @bitwarden/dept-bre + +## These are shared workflows ## +.github/workflows/cleanup-after-pr.yml +.github/workflows/cleanup-rc-branch.yml +.github/workflows/release.yml +.github/workflows/version-bump.yml + # Database Operations for database changes src/Sql/** @bitwarden/dept-dbops util/EfShared/** @bitwarden/dept-dbops @@ -50,6 +65,3 @@ src/Admin/Views/Tools @bitwarden/team-billing-dev # Multiple owners - DO NOT REMOVE (BRE) **/packages.lock.json Directory.Build.props - -# BRE for release workflow changes -.github/workflows/release.yml @bitwarden/dept-bre From 3cc40b93d22cb5659828187fbbf93ad4339dab58 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Ch=C4=99ci=C5=84ski?= Date: Thu, 17 Oct 2024 11:30:25 +0200 Subject: [PATCH 3/3] Update .github/CODEOWNERS Co-authored-by: Vince Grassia <593223+vgrassia@users.noreply.github.com> --- .github/CODEOWNERS | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 513ec7c42c43..80622686f451 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -14,10 +14,12 @@ .github/workflows/publish.yml @bitwarden/dept-bre ## These are shared workflows ## +.github/workflows/_move_finalization_db_scripts.yml +.github/workflows/build.yml .github/workflows/cleanup-after-pr.yml .github/workflows/cleanup-rc-branch.yml .github/workflows/release.yml -.github/workflows/version-bump.yml +.github/workflows/repository-management.yml # Database Operations for database changes src/Sql/** @bitwarden/dept-dbops