Skip to content

Add update-keys.sh for nodejs release keys update - #1297

Merged
PeterDaveHello merged 1 commit into
nodejs:masterfrom
PeterDaveHello:add-update-keys-script
Jul 23, 2020
Merged

Add update-keys.sh for nodejs release keys update#1297
PeterDaveHello merged 1 commit into
nodejs:masterfrom
PeterDaveHello:add-update-keys-script

Conversation

@PeterDaveHello

@PeterDaveHelloPeterDaveHello commented Jul 22, 2020

Copy link
Copy Markdown
Member

@SimenBSimenB 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.

Would be great if it was a separate file rather than crawling the readme. Nice to have a script, tho!

@nschonninschonni 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.

Don't know awk well enough, but this looks right

Comment threadupdate-keys.sh Outdated
@PeterDaveHello
PeterDaveHelloforce-pushed the add-update-keys-script branch from e551649 to ae84d3fCompareJuly 22, 2020 19:12
@PeterDaveHello
PeterDaveHelloforce-pushed the add-update-keys-script branch from ae84d3f to db2fe72CompareJuly 22, 2020 19:14
@PeterDaveHello
PeterDaveHello requested a review from a teamJuly 22, 2020 19:14
@PeterDaveHello

Copy link
Copy Markdown
MemberAuthor

@nodejs/docker I did a few minor changes, please help take a look, thanks!

@nschonninschonni 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.

LGTM. It may be worth making the keys a version specific file and pull from the individual branches later

@PeterDaveHello
PeterDaveHello merged commit 485333c into nodejs:masterJul 23, 2020
@PeterDaveHello
PeterDaveHello deleted the add-update-keys-script branch July 23, 2020 11:47
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@PeterDaveHello@nschonni@SimenB