Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add localization team approvers (ar bn es pt ko) #432

Merged
merged 1 commit into from
Feb 17, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
61 changes: 59 additions & 2 deletions .github/settings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,13 +38,19 @@ collaborators:
- username: Eviekim
permission: push

- username: yunkon-kim
permission: push

# l10n pt approvers
- username: edsoncelio
permission: push

- username: brunoguidone
permission: push

- username: jessicalins
permission: push

# l10n hi approvers
- username: Garima-Negi
permission: push
Expand Down Expand Up @@ -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:

Expand Down Expand Up @@ -119,6 +151,7 @@ branches:
- seokho-son
- Eviekim
- jihoon-seo
- yunkon-kim
teams: []
enforce_admins: null
required_linear_history: null
Expand All @@ -136,6 +169,7 @@ branches:
users:
- edsoncelio
- brunoguidone
- jessicalins
teams: []
enforce_admins: null
required_linear_history: null
Expand Down Expand Up @@ -207,7 +241,10 @@ branches:
apps: []
# ar approvers
users:
- seokho-son
- TarekMSayed
- same7ammar
- AShabana
- hacktron95
teams: []
enforce_admins: null
required_linear_history: null
Expand All @@ -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
11 changes: 7 additions & 4 deletions CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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