doc: document the @nodejs/doc-kit subteam #176
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,6 +17,12 @@ | |
| - [@mikeesto](https://github.com/mikeesto) - **Michael Esteban** (he/him) | ||
| - [@ovflowd](https://github.com/ovflowd) - **Claudio Wunder** (they/them) | ||
|
|
||
| ## Node.js Doc Kit Team (`@nodejs/doc-kit`) | ||
|
|
||
| - [@avivkeller](https://github.com/avivkeller) - **Aviv Keller** (he/him) | ||
| - [@bmuenzenmeyer](https://github.com/bmuenzenmeyer) - **Brian Muenzenmeyer** (he/him) | ||
| - [@ovflowd](https://github.com/ovflowd) - **Claudio Wunder** (they/them) | ||
|
|
||
| ## Node.js Web Infra Team (`@nodejs/web-infra`) | ||
|
|
||
| - [@avivkeller](https://github.com/avivkeller) - **Aviv Keller** <<me@aviv.sh>> (he/him) | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -2,34 +2,34 @@ | |||||
|
|
||||||
| ## Repositories | ||||||
|
|
||||||
| | Repository | Everyone | @nodejs/web | @nodejs/nodejs-website | @nodejs/doc-kit | @nodejs/web-infra | @nodejs/web-admins | Notes | | ||||||
| | ---------------------------------------- | -------- | ----------- | ---------------------- | --------------- | ----------------- | ------------------ | ----------------------------------------------------------------------------------------------------------- | | ||||||
| | **[nodejs/discord-status-worker][]** | Read | - | - | - | Maintain | Admin | | | ||||||
| | **[nodejs/doc-kit][]** | Read | - | Write | Maintain | Maintain | Admin | | | ||||||
|
Contributor
There was a problem hiding this comment. the doc-kit team shouldn't be 
Suggested change
Member
Author
There was a problem hiding this comment. I was wondering if they should be |
||||||
| | **[nodejs/learn][]** | Read | - | Write | - | Maintain | Admin | | | ||||||
| | **[nodejs/node.js.org][]** | Read | Write | - | - | Maintain | Admin | | | ||||||
| | **[nodejs/nodejs.dev][]** | Read | - | Write | - | Maintain | Admin | This repository is _archived_. | | ||||||
| | **[nodejs/nodejs.org][]** | Read | - | Write | - | Maintain | Admin | | | ||||||
| | **[nodejs/nodejs.org-archive][]** | Read | Write | - | - | Maintain | Admin | This repository is _archived_. | | ||||||
| | **[nodejs/release-cloudflare-worker][]** | Read | - | - | - | Maintain | Admin | | | ||||||
| | **[nodejs/web-team][]** | Read | Triage | - | - | - | Admin | | | ||||||
| | **[nodejs/website-cloudflare-worker][]** | - | - | - | - | Maintain | Admin | | | ||||||
| | **[nodejs/undici-website][]** | Read | - | - | - | Maintain | - | | | ||||||
|
|
||||||
| ## External Services | ||||||
|
|
||||||
| | Service | Everyone | @nodejs/web | @nodejs/nodejs-website | @nodejs/doc-kit | @nodejs/web-infra | @nodejs/web-admins | Notes | | ||||||
| | ---------------------------- | ---------------------------------- | ----------- | ---------------------- | --------------- | ----------------- | ------------------ | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | ||||||
| | **[1Password][]** | - | - | - | - | Admin | Admin | | | ||||||
| | **[Atlassian Statuspage][]** | [Read](https://status.nodejs.org/) | - | - | - | App Admin | Org Admin | | | ||||||
| | **[Chromatic][]** | Read | - | Write | - | Admin | Admin | Access to this service is granted via GitHub authentication. | | ||||||
| | **[Cloudflare][]** | - | - | - | - | Read | Admin | Access to this service is controlled by @nodejs/build. Additional access may be granted on a case-by-case basis. | | ||||||
| | **[Codecov][]** | Read | - | Admin | - | Admin | Admin | Access to this service is granted via GitHub authentication, and only related to the repository above. | | ||||||
| | **[Crowdin][]** | Read | - | - | - | Admin | Admin | Credentials for a user with elevated exist in 1Password. Access to this service may be granted to outside collaborators on a case-by-case, language-by-language basis. | | ||||||
|
ovflowd marked this conversation as resolved.
|
||||||
| | **[Figma][]** | Read | - | Write | - | - | - | Access to this service is controlled by the OpenJS Foundation and @avivkeller, and only covers website-related design files. | | ||||||
| | **[HackMD][]** | - | - | - | - | - | Admin | | | ||||||
| | **[Sentry][]** | - | - | - | - | Manager | Owner | | | ||||||
| | **[Vercel][]** | - | - | - | - | Developer | Admin | Along with individual access, credentials for a user with elevated exist in 1Password. | | ||||||
|
|
||||||
| ## Access Tokens & Automations | ||||||
|
|
||||||
|
|
||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -4,9 +4,10 @@ This repository contains the governance documentation, workflows, and shared res | |||||
|
|
||||||
| ## Sub-Team Structure | ||||||
|
|
||||||
| The Node.js Web Team consists of three specialized subteams: | ||||||
|
|
||||||
| - **Website Team** ([`@nodejs/nodejs-website`](https://github.com/orgs/nodejs/teams/nodejs-website)) - Day-to-day website development | ||||||
| - **Doc Kit Team** ([`@nodejs/doc-kit`](https://github.com/orgs/nodejs/teams/doc-kit)) - Day-to-day development of [nodejs/doc-kit](https://github.com/nodejs/doc-kit) | ||||||
|
Member
There was a problem hiding this comment.
Suggested change
|
||||||
| - **Web Infra Team** ([`@nodejs/web-infra`](https://github.com/orgs/nodejs/teams/web-infra)) - Infrastructure and CI/CD | ||||||
|
|
||||||
| _There is also the **Web Admins Team** ([`@nodejs/web-admins`](https://github.com/orgs/nodejs/teams/web-admins)), which is responsible for administrative operations relating to the Web Team._ | ||||||
|
|
||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,19 @@ | ||||||||||||||||||
| # @nodejs/doc-kit Onboarding | ||||||||||||||||||
|
|
||||||||||||||||||
| This document is an outline of the tasks necessary to onboard a new member of the Doc Kit team (@nodejs/doc-kit). | ||||||||||||||||||
|
Member
There was a problem hiding this comment.
Suggested change
|
||||||||||||||||||
|
|
||||||||||||||||||
| Please note that these tasks should only be performed after an individual's nomination has passed. | ||||||||||||||||||
| To start the nomination process, see [GOVERNANCE.md](../GOVERNANCE.md). | ||||||||||||||||||
|
|
||||||||||||||||||
| ## Tasks | ||||||||||||||||||
|
|
||||||||||||||||||
| - [ ] Add the **Nominee** to the [`@nodejs/doc-kit`](https://github.com/orgs/nodejs/teams/doc-kit) team on GitHub. | ||||||||||||||||||
| - [ ] An OpenJS Slack Admin should add their Slack account to the `nodejs-website-team` team. | ||||||||||||||||||
|
ovflowd marked this conversation as resolved.
|
||||||||||||||||||
| - [ ] Add the **Nominee** to the `#nodejs-website` OpenJS Slack channel. | ||||||||||||||||||
|
Comment on lines
+10
to
+12
Member
There was a problem hiding this comment.
Suggested change
|
||||||||||||||||||
|
|
||||||||||||||||||
| ### Recommendations | ||||||||||||||||||
|
|
||||||||||||||||||
| - [ ] The **Nominee** should have some form of physical 2FA (i.e. Yubikey) or passkey enabled on their GitHub account. | ||||||||||||||||||
| - [ ] The **Nominee** should have any SSH or GPG key attached to their GitHub account have passphrases and/or be stored on a physical 2FA device. | ||||||||||||||||||
|
ovflowd marked this conversation as resolved.
|
||||||||||||||||||
| - [ ] The **Nominee** should sign their Git commits. | ||||||||||||||||||
|
Comment on lines
+14
to
+18
Member
There was a problem hiding this comment. This is only for web-infra, right?
Member
There was a problem hiding this comment. Everyone should be doing these things, they improve our security posture. I think web-infra must do these things?
Member
Author
There was a problem hiding this comment. Agree, I think we could leave as is, but also fine either removing or copy pasting this to the other team-specific recommendations. |
||||||||||||||||||
|
|
||||||||||||||||||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.