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 Hindi approvers in CODEOWNERS #298

Closed

Conversation

jayesh-srivastava
Copy link
Collaborator

A task based on the issue #297 (Initiate Hindi Localization Team)

@netlify
Copy link

netlify bot commented Jan 8, 2022

✔️ Deploy Preview for cncfglossary ready!

🔨 Explore the source changes: c49b4e0

🔍 Inspect the deploy log: https://app.netlify.com/sites/cncfglossary/deploys/61d959a4e938990007b851b5

😎 Browse the preview: https://deploy-preview-298--cncfglossary.netlify.app

@anubha-v-ardhan
Copy link
Contributor

We also need to add a block for Hindi in config.toml

@CathPag CathPag requested a review from seokho-son January 8, 2022 22:47
Copy link
Collaborator

@CathPag CathPag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CathPag
Copy link
Collaborator

CathPag commented Jan 10, 2022

As @seokho-son mentioned on Slack, we are still testing the process with the Korean team, so we'll hold on merging this until that is the process is confirmed as working well.

@CathPag CathPag added the hold Wait, please do not proceed this yet label Jan 11, 2022
Copy link
Contributor

@anubha-v-ardhan anubha-v-ardhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jayesh-srivastava I believe you can close this PR as this change is being done by #342 for all localizations

@jihoon-seo jihoon-seo added the lang/hi for Hindi label Jan 25, 2022
@seokho-son
Copy link
Collaborator

Let me close this PR :)
codeowners will be handled by #342

@seokho-son seokho-son closed this Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold Wait, please do not proceed this yet lang/hi for Hindi
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants