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

Update cncf/glossary external_collaborators #248

Merged
merged 1 commit into from
Jun 16, 2023

Conversation

seokho-son
Copy link
Contributor

Hello folks !
I am one of the maintainers of the https://github.com/cncf/glossary

We hope to update permissions for the cncf/glossary repository based on..

  1. Add Turkish approvers (external_collaborators with write) Initiate Turkish l10n team glossary#2123
    Add: @aliok
    Add: @halil-bugol
    Add: @developer-guy
    Add: @eminalemdar

  2. Replace Bengali localization approvers. Update CODEOWNERS for Bengali branch glossary#2100
    Remove: @Mouly22 @ikramulkayes
    Add: @Arindam200 @sajibAdhi

  3. Re-Add a localization approver (I don't know why but an approver has been removed from the list)
    Add: @Jacob953

Signed-off-by: Seokho Son <shsongist@gmail.com>
@clowarden
Copy link

clowarden bot commented Jun 16, 2023

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Github

  • user ikramulkayes is no longer a collaborator of repository glossary
  • user Mouly22 is no longer a collaborator of repository glossary
  • user Arindam200 is now a collaborator (role: write) of repository glossary
  • user sajibAdhi is now a collaborator (role: write) of repository glossary
  • user eminalemdar is now a collaborator (role: write) of repository glossary
  • user aliok is now a collaborator (role: write) of repository glossary
  • user Jacob953 is now a collaborator (role: write) of repository glossary
  • user halil-bugol is now a collaborator (role: write) of repository glossary
  • user developer-guy is now a collaborator (role: write) of repository glossary

🔸 Please review the changes detected as they will be applied immediately once this PR is merged 🔸

@cjyabraham cjyabraham merged commit 400a7f7 into cncf:main Jun 16, 2023
3 of 4 checks passed
@clowarden
Copy link

clowarden bot commented Jun 16, 2023

Reconciliation completed

✅ The reconciliation completed successfully and all changes have been applied across the services!

Changes applied

Github

  • user Mouly22 is no longer a collaborator of repository glossary
  • user ikramulkayes is no longer a collaborator of repository glossary
  • user sajibAdhi is now a collaborator (role: write) of repository glossary
  • user halil-bugol is now a collaborator (role: write) of repository glossary
  • user Jacob953 is now a collaborator (role: write) of repository glossary
  • user eminalemdar is now a collaborator (role: write) of repository glossary
  • user developer-guy is now a collaborator (role: write) of repository glossary
  • user aliok is now a collaborator (role: write) of repository glossary
  • user Arindam200 is now a collaborator (role: write) of repository glossary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants