FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: adding Italo A. Casas PGP Fingerprint by italoacasas · Pull Request #11202 · nodejs/node · GitHub

/ node Public

doc: adding Italo A. Casas PGP Fingerprint - #11202

Closed
italoacasas wants to merge 1 commit into
nodejs:masterfrom
italoacasas:readme/key-fingerprint
Closed

doc: adding Italo A. Casas PGP Fingerprint#11202
italoacasas wants to merge 1 commit into
nodejs:masterfrom
italoacasas:readme/key-fingerprint

Conversation

Copy link
Copy Markdown

Adding my PGP Fingerprint to the release team.

CTC vote: nodejs/CTC#48

I have been doing the first part of the release for the last 2 or 3 releases in the current branch (7.x), this week release (7.6.0) is going to be a small one and is going to be easier for me to do the complete release. I ask Evan Lucas in IRC and he is going to help me in the process.

Pending Items:

  • Be accepted in the release team in Github (I request access already)
  • Access to the release CI
  • Merge this PR

cc: @nodejs/release @evanlucas

nodejs-github-bot added the doc Issues and PRs related to the documentations. label Feb 6, 2017

bnoordhuis commented Feb 6, 2017
edited
Loading

Copy link
Copy Markdown
Member

Have you uploaded your (EDIT: public) key to https://sks-keyservers.net/? I can't seem to find you.

italoacasas commented Feb 6, 2017
edited
Loading

Copy link
Copy Markdown
Author

@bnoordhuis I did it but seems like is not working :(, working on that.

Fixed.

evanlucas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM, but maybe wait to get signoff from a few other members of @nodejs/release since there are some additional steps to be taken (like getting your public key for promoting the binaries)

jasnell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM once all the necessary other steps are complete :-)

MylesBorins left a comment
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Can you add the key to the list below (line 378) that includes that command to import the key from the keyserver.

Once this is done I give permission to anyone to change my review to approved

Copy link
Copy Markdown
Contributor

I've verified that gpg --keyserver pool.sks-keyservers.net --recv-keys 56730D5401028683275BD3C23EFEFE93C4CFFFE works as expected. I have also taken the liberty of signing the key and sending it to the public key server

italoacasas force-pushed the readme/key-fingerprint branch from 09a1d15 to cdda682 Compare February 7, 2017 22:28

Copy link
Copy Markdown
Author

I add the command as requested. Thanks, everyone.

jasnell pushed a commit that referenced this pull request Feb 11, 2017
PR-URL: #11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>

jasnell commented Feb 11, 2017

Copy link
Copy Markdown
Member

Landed in c8fff70

jasnell closed this Feb 11, 2017
italoacasas pushed a commit that referenced this pull request Feb 13, 2017
PR-URL: #11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Feb 14, 2017
PR-URL: nodejs#11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
italoacasas deleted the readme/key-fingerprint branch February 20, 2017 01:30
italoacasas restored the readme/key-fingerprint branch February 20, 2017 01:30
krydos pushed a commit to krydos/node that referenced this pull request Feb 25, 2017
PR-URL: nodejs#11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
jasnell pushed a commit that referenced this pull request Mar 7, 2017
PR-URL: #11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
jasnell pushed a commit that referenced this pull request Mar 7, 2017
PR-URL: #11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Mar 9, 2017
PR-URL: #11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins mentioned this pull request Mar 9, 2017
MylesBorins pushed a commit that referenced this pull request Mar 9, 2017
PR-URL: #11202
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins mentioned this pull request Mar 9, 2017
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL