Skip to content

doc: edit deprecation section of Collaborator Guide - #26419

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:copyedit-deprecations
Closed

doc: edit deprecation section of Collaborator Guide#26419
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:copyedit-deprecations

Conversation

@Trott

@TrottTrott commented Mar 4, 2019

Copy link
Copy Markdown
Member

Edit the last few paragraphs of the Collaborator Guide section on
deprecations.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Edit the last few paragraphs of the Collaborator Guide section on
deprecations.
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Mar 4, 2019
@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 4, 2019
@vsemozhetbytvsemozhetbyt added meta Issues and PRs related to the general management of the project. deprecations Issues and PRs related to deprecations. labels Mar 4, 2019
Comment threadCOLLABORATOR_GUIDE.md
* If used with the [`--throw-deprecation`][] flag, will throw a runtime error.

* End-of-life
* End-of-Life

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.

I thought we'd added a lint rule for this, but we forbade End-Of-Life. A quick search shows End-of-life also appears in BUILDING.md, so nodejs/remark-preset-lint-node#13.

@Trott

Trott commented Mar 6, 2019

Copy link
Copy Markdown
MemberAuthor

Landed in 05606c6

@TrottTrott closed this Mar 6, 2019
Trott added a commit to Trott/io.js that referenced this pull request Mar 6, 2019
Edit the last few paragraphs of the Collaborator Guide section on
deprecations.
PR-URL: nodejs#26419
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request Mar 12, 2019
Edit the last few paragraphs of the Collaborator Guide section on
deprecations.
PR-URL: nodejs#26419
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
BridgeAR pushed a commit that referenced this pull request Mar 14, 2019
Edit the last few paragraphs of the Collaborator Guide section on
deprecations.
PR-URL: #26419
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
@Trott
Trott deleted the copyedit-deprecations branch January 13, 2022 22:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.deprecationsIssues and PRs related to deprecations.docIssues and PRs related to the documentations.metaIssues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@Trott@nodejs-github-bot@jasnell@targos@richardlau@BethGriggs@BridgeAR@vsemozhetbyt