-
Notifications
You must be signed in to change notification settings - Fork 192
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 checklist to update maintainers team #2584
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: jfy133 <[email protected]>
✅ Deploy Preview for nf-core ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@edmundmiller I don't seem to be able to find the page (either by clicking through the website or guessing the URL), any ideas? |
Oh, wait found it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but I sort of wonder if it makes sense to have this page exposed in the docs overview page... It's only basically for 3 people +maybe the maintainers team in an an emergency (technically)
subtitle: A step-by-step guide for changing of the guard | ||
markdownPlugin: checklist | ||
--- | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Roughly once a year we review the maintainers team to ensure we have sufficient capacity to keep up with the community activity. | |
The following steps are what the maintainers team leads need to do during the review. | |
2. [ ] Ask for nominations in maintainers meeting, discuss and approve | ||
3. [ ] Ask nominees | ||
4. [ ] Once accepted, add to [#team-maintainers](https://nfcore.slack.com/archives/C043UU89KKQ) (and remove alumni/extras) | ||
5. [ ] Once accepted, make PRs and merge in: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5. [ ] Once accepted, make PRs and merge in: | |
5. [ ] Once accepted, make PRs to add/move to alumni members and merge in: |
Yeah, had the same feeling... |
Can we leave the file in the same place but not index it in the docs overview section @mashehu ? Like 'hidden: true' in the header or something? |
We don't have that option, not sure if we really want to have hidden files... |
Hidden only from being displayed in the overview, not that the page isn't accessible at all (in case I wasn't clear) |
Yeah, I agree with this. I don't think there's anything sensitive in there, but it would just be clutter that most people won't ever need. |
@netlify /docs/checklists/maintainers_team_updates