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

Org maintainers update #1703

Merged
merged 2 commits into from
Apr 16, 2024

Conversation

toddysm
Copy link
Contributor

@toddysm toddysm commented Apr 3, 2024

@yizha1 yizha1 requested review from a team April 8, 2024 23:53
Copy link

@priteshbandi priteshbandi left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -0,0 +1,3 @@
# Owners (in alphabetical order)
# Note: This is only for the notaryproject/notary repo and includes the Org level maintainers
* @hukeping @jonnystoten @niazfk @priteshbandi @shizhMSFT @toddysm @vaninrao10 @yizha1
Copy link
Member

@FeynmanZhou FeynmanZhou Apr 9, 2024

Choose a reason for hiding this comment

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

For inactive repos without activities neither active repo maintainers in recent 1~2 years, should we prioritize the archival process instead? There is an issue notaryproject/.github#70 already proposed archive the notary repo.

This will relax unnecessary workloads to org maintainers and avoid confusing new users going forward. We add new maintainers to those repos indicate that new maintainers are responsible for its continuous maintenance for users. FYI the archival process below from the archival process doc:

The repository-level maintainers should be noticed about this archiviation issue. Once the proposal issue has a supermajority (two-thirds) of approval from Notary Project governance maintainers and there is no response from the repository-level maintainers after 30 days notice, the repository can be archived.”

Copy link
Contributor

@yizha1 yizha1 left a comment

Choose a reason for hiding this comment

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

LGTM. I would suggest merging this PR to refresh the latest org maintainers, and tracking the archiving issue at notaryproject/.github#70

@yizha1 yizha1 merged commit 9d2b3b3 into notaryproject:master Apr 16, 2024
1 of 5 checks passed
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.

4 participants