Skip to content

doc: copyedit releases.md - #4384

Closed
Trott wants to merge 2 commits into
nodejs:masterfrom
Trott:releases-copyedit
Closed

doc: copyedit releases.md#4384
Trott wants to merge 2 commits into
nodejs:masterfrom
Trott:releases-copyedit

Conversation

@Trott

@TrottTrott commented Dec 22, 2015

Copy link
Copy Markdown
Member

No description provided.

@TrottTrott added the doc Issues and PRs related to the documentations. label Dec 22, 2015
@jasnell

jasnell commented Dec 22, 2015

Copy link
Copy Markdown
Member

LGTM

bnoordhuis
bnoordhuis reviewed Dec 22, 2015
View reviewed changes
Comment threaddoc/releases.md Outdated

@bnoordhuisbnoordhuisDec 22, 2015

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.

s/SCP/scp/?

@TrottTrottDec 22, 2015

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

In general, I believe SCP is used for the protocol and scp for the command, similar to SSH vs. ssh. I'm fine with either here. No strong feelings either way.

@rvaggrvaggDec 23, 2015

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.

the intention was for it to refer to the command, if you do it as scp it should be clear

@TrottTrottDec 23, 2015

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

OK, SCP -> scp fixup pushed.

@bnoordhuis

bnoordhuis commented Dec 22, 2015

Copy link
Copy Markdown
Member

LGTM

1 similar comment
@cjihrig

cjihrig commented Dec 22, 2015

Copy link
Copy Markdown
Contributor

LGTM

@rvagg

rvagg commented Dec 23, 2015

Copy link
Copy Markdown
Member

Changes lgtm, thanks for the effort here @Trott.

The doc is still out of date though, I'd love to go through and update it but I'm not sure I'll be able to find the time for a while. If any of the @nodejs/release team who have done a release recently want to have a go it'd be greatly appreciated. There are some things that stand out immediately:

  • iojs+release options have expanded and need explaining
  • tools/release.sh has a -s argument that can be used to re-sign a build where the signing failed for some reason, I don't think that's mentioned in there
  • details on how to publish a blog post
  • the extra descriptors we put in summaries, tags and other places now: "(Stable)", "(LTS)", "(Maintenance)"
  • having a PR-URL for a commit message is preferred
  • does it mention that you should push your -proposal branch to this repo so others in the release team can take over? that feels like a more recent evolution of our process.

I'm sure there are other things.

@Trott

Trott commented Dec 23, 2015

Copy link
Copy Markdown
MemberAuthor

@rvagg Safe for me to interpret your comment as a wish list of updates you'd like to see someone do soon if someone knowledgable can get to it, but no reason to refrain from merging the changes that are here at this time?

@rvagg

rvagg commented Dec 24, 2015

Copy link
Copy Markdown
Member

@Trott yes, that's correct, this lgtm as is for a simple copyedit, great job.

Trott added a commit to Trott/io.js that referenced this pull request Dec 24, 2015
@Trott
doc: copyedit releases.md
b094402
PR-URL: nodejs#4384
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@Trott

Trott commented Dec 24, 2015

Copy link
Copy Markdown
MemberAuthor

Landed in b094402

@TrottTrott closed this Dec 24, 2015
Trott added a commit that referenced this pull request Dec 30, 2015
@TrottMyles Borins
doc: copyedit releases.md
956facb
PR-URL: #4384
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@cjihrigcjihrig mentioned this pull request Jan 5, 2016
Closed
Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request Jan 6, 2016
@Trott@Fishrock123
doc: copyedit releases.md
e79eda7
PR-URL: nodejs#4384
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 19, 2016
@TrottMyles Borins
doc: copyedit releases.md
2b1804f
PR-URL: #4384
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Jan 19, 2016
Merged
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
@TrottMichael Scovetta
doc: copyedit releases.md
d449e35
PR-URL: nodejs#4384
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@Trott
Trott deleted the releases-copyedit branch January 13, 2022 22:31
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

docIssues and PRs related to the documentations.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@jasnell@bnoordhuis@cjihrig@rvagg@MylesBorins