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

Removing Catherine maintainer mentions #2268

Merged
merged 3 commits into from
Sep 5, 2023
Merged

Removing Catherine maintainer mentions #2268

merged 3 commits into from
Sep 5, 2023

Conversation

CathPag
Copy link
Collaborator

@CathPag CathPag commented Aug 1, 2023

Describe your changes

Removing Catherine as maintainer mentions

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".

Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Aug 1, 2023

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit fefabec
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/64f7780e61ec94000776e1fd
😎 Deploy Preview https://deploy-preview-2268--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.

@github-actions github-actions bot added the lang/en for English label Aug 1, 2023
Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
@CathPag CathPag changed the title Removing Catherine as maintainer mentions Removing Catherine maintainer mentions Aug 1, 2023
@CathPag CathPag added the maintainers Use this label if PR requires maintainers to take action label Aug 1, 2023
@iamNoah1
Copy link
Collaborator

@jihoon-seo @seokho-son don't we need to remove her also from the owners file(s)?

@nate-double-u
Copy link
Member

@jihoon-seo @seokho-son don't we need to remove her also from the owners file(s)?

I think so, but we can open a follow up issue.

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 @CathPag

content/en/_index.md Outdated Show resolved Hide resolved
Signed-off-by: Nate W <natew@cncf.io>
@nate-double-u nate-double-u merged commit 10d0408 into main Sep 5, 2023
@nate-double-u nate-double-u deleted the CathPag-patch-1 branch September 5, 2023 18:56
@nate-double-u
Copy link
Member

It looks like #2343 takes care of the owners file.

Fionn88 pushed a commit to Fionn88/glossary that referenced this pull request Sep 15, 2023
* Removing Catherine as maintainer 

Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>

* how to

Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>

* Update content/en/_index.md

Signed-off-by: Nate W <natew@cncf.io>

---------

Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
Signed-off-by: Nate W <natew@cncf.io>
Co-authored-by: Nate W <natew@cncf.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/en for English maintainers Use this label if PR requires maintainers to take action
Projects
Development

Successfully merging this pull request may close these issues.

3 participants