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

Replace @pichuang with @tico88612 as zh-tw branch approver #2960

Merged
merged 1 commit into from
Mar 9, 2024

Conversation

pichuang
Copy link
Contributor

Describe your changes

Suggest that @pichuang 's own permissions be transferred to @tico88612 , who can contribute more to the glossary project

Related issue number or link (ex: resolves #issue-number)

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Copy link

netlify bot commented Feb 26, 2024

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 298f9af
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/65dc82deb7371c000813190a
😎 Deploy Preview https://deploy-preview-2960--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Phil Huang <phil.huang@microsoft.com>
@pichuang pichuang force-pushed the replace-zh-tw-approver branch from 6ef6589 to 298f9af Compare February 26, 2024 12:23
@pichuang
Copy link
Contributor Author

Based on the past record of actual contribution to zh-tw, @tico88612 is recommended to be the reviewer of the project, in the hope that the project can move forward more continuously.

Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

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

Thanks for all your work @pichuang, and welcome to the team @tico88612!

@iamNoah1
Copy link
Collaborator

iamNoah1 commented Mar 9, 2024

Thanks @pichuang and welcome @tico88612

@iamNoah1 iamNoah1 merged commit 88f5f22 into cncf:main Mar 9, 2024
6 checks passed
@tico88612
Copy link
Collaborator

Hi @nate-double-u @iamNoah1
I still can not assign an issue or edit the PR. Could you help me? Thanks.

image

@seokho-son
Copy link
Collaborator

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.

5 participants