-
Notifications
You must be signed in to change notification settings - Fork 577
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
docs: add GPG validation onboarding step #966
Conversation
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 with a tiny suggestion.
@@ -150,6 +150,8 @@ After the nominee's first prepared release has been promoted the new releaser mu | |||
* Be added to the GitHub [releasers team](https://github.com/orgs/nodejs/teams/releasers) in the Node.js org (grants ci-release access) | |||
* Be added to the GitHub [security-release team](https://github.com/orgs/nodejs/teams/security-release) in the Node.js and nodejs-private orgs | |||
* Have a single, high quality SSH key added to the "dist" user on the primary www server (see below for guidelines regarding SSH key quality) | |||
* Add your GPG key to your GitHub account for verification. [Github Guide](https://docs.github.com/en/authentication/managing-commit-signature-verification/adding-a-gpg-key-to-your-github-account). | |||
* Check that the key is added in your profile: `https://github.com/your-username.gpg` |
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.
Can we check it using gh
? So in the future, we could integrate it on @node-core/utils.
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.
Yeah! 100% possible I think as this is not a protected endpoint. Let me see If I can create a PR for it
docs/gpg-validation |
@@ -150,6 +150,8 @@ After the nominee's first prepared release has been promoted the new releaser mu | |||
* Be added to the GitHub [releasers team](https://github.com/orgs/nodejs/teams/releasers) in the Node.js org (grants ci-release access) | |||
* Be added to the GitHub [security-release team](https://github.com/orgs/nodejs/teams/security-release) in the Node.js and nodejs-private orgs | |||
* Have a single, high quality SSH key added to the "dist" user on the primary www server (see below for guidelines regarding SSH key quality) | |||
* Add your GPG key to your GitHub account for verification. [Github Guide](https://docs.github.com/en/authentication/managing-commit-signature-verification/adding-a-gpg-key-to-your-github-account). | |||
* Check that the key is added in your profile: `https://github.com/your-username.gpg` |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
@@ -150,6 +150,8 @@ After the nominee's first prepared release has been promoted the new releaser mu | |||
* Be added to the GitHub [releasers team](https://github.com/orgs/nodejs/teams/releasers) in the Node.js org (grants ci-release access) | |||
* Be added to the GitHub [security-release team](https://github.com/orgs/nodejs/teams/security-release) in the Node.js and nodejs-private orgs | |||
* Have a single, high quality SSH key added to the "dist" user on the primary www server (see below for guidelines regarding SSH key quality) | |||
* Add your GPG key to your GitHub account for verification. [Github Guide](https://docs.github.com/en/authentication/managing-commit-signature-verification/adding-a-gpg-key-to-your-github-account). | |||
* Check that the key is added in your profile: `https://github.com/your-username.gpg` | |||
* Open a PR to add their GPG key to the nodejs/node [README.md](https://github.com/nodejs/node/#release-keys) | |||
* Once landed, the key should be cherry-picked to all active release staging branches (i.e. the active `vMM.x-staging` branches). | |||
* Any future updates to the key should also be cherry-picked to all active release staging branches. |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
I will merge it, as seems that we are on the same page and it has been open for a while 👍 |
Related: #965