From 12a216279082540d27ce50f7ad6b81d98f2a2f38 Mon Sep 17 00:00:00 2001 From: Seokho Son Date: Sun, 13 Feb 2022 02:00:18 +0900 Subject: [PATCH] Add localization team approvers (ar bn es pt ko) --- .github/settings.yml | 61 ++++++++++++++++++++++++++++++++++++++++++-- CODEOWNERS | 11 +++++--- 2 files changed, 66 insertions(+), 6 deletions(-) diff --git a/.github/settings.yml b/.github/settings.yml index 90589d404c..a9d46b0671 100644 --- a/.github/settings.yml +++ b/.github/settings.yml @@ -38,6 +38,9 @@ collaborators: - username: Eviekim permission: push + - username: yunkon-kim + permission: push + # l10n pt approvers - username: edsoncelio permission: push @@ -45,6 +48,9 @@ collaborators: - username: brunoguidone permission: push + - username: jessicalins + permission: push + # l10n hi approvers - username: Garima-Negi permission: push @@ -80,9 +86,35 @@ collaborators: permission: push # l10n ar approvers + - username: TarekMSayed + permission: push + + - username: same7ammar + permission: push + + - username: AShabana + permission: push + + - username: hacktron95 + permission: push # l10n bn approvers + - username: mitul3737 + permission: push + - username: Mouly22 + permission: push + + - username: ikramulkayes + permission: push + + # l10n es approvers + # Note: CathPag is both a maintainer (maintain) and de approver (push) + - username: raelga + permission: push + + - username: electrocucaracha + permission: push branches: @@ -119,6 +151,7 @@ branches: - seokho-son - Eviekim - jihoon-seo + - yunkon-kim teams: [] enforce_admins: null required_linear_history: null @@ -136,6 +169,7 @@ branches: users: - edsoncelio - brunoguidone + - jessicalins teams: [] enforce_admins: null required_linear_history: null @@ -207,7 +241,10 @@ branches: apps: [] # ar approvers users: - - seokho-son + - TarekMSayed + - same7ammar + - AShabana + - hacktron95 teams: [] enforce_admins: null required_linear_history: null @@ -223,7 +260,27 @@ branches: apps: [] # bn approvers users: - - seokho-son + - mitul3737 + - Mouly22 + - ikramulkayes + teams: [] + enforce_admins: null + required_linear_history: null + + # l10n branch for es contents only + - name: dev-es + protection: + required_pull_request_reviews: + required_approving_review_count: 2 + require_code_owner_reviews: true + required_status_checks: null + restrictions: + apps: [] + # es approvers + users: + - CathPag + - raelga + - electrocucaracha teams: [] enforce_admins: null required_linear_history: null diff --git a/CODEOWNERS b/CODEOWNERS index 658759461a..71e72431e0 100644 --- a/CODEOWNERS +++ b/CODEOWNERS @@ -10,10 +10,10 @@ # in each `/content/language/` directory. # Owners of Korean contents -/content/ko/ @seokho-son @Eviekim @jihoon-seo +/content/ko/ @seokho-son @Eviekim @jihoon-seo @yunkon-kim # Owners of Portuguese contents -/content/pt/ @edsoncelio @brunoguidone +/content/pt/ @edsoncelio @brunoguidone @jessicalins # Owners of Hindi contents /content/hi/ @Garima-Negi @sayantani11 @anubha-v-ardhan @jayesh-srivastava @@ -25,7 +25,10 @@ /content/it/ @Giulia-dipietro @meryem-ldn @annalisag-spark @sistella # Owners of Arabic contents -# /content/ar/ @TarekMSayed @same7ammar @AShabana @hacktron95 +/content/ar/ @TarekMSayed @same7ammar @AShabana @hacktron95 # Owners of Bengali contents -# /content/bn/ @mitul3737 @Mouly22 @ikramulkayes +/content/bn/ @mitul3737 @Mouly22 @ikramulkayes + +# Owners of Spanish contents +/content/es/ @CathPag @raelga @electrocucaracha