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

doc: add missing deprecation number by bcoe · Pull Request #35630 · nodejs/node · GitHub

/ node Public

doc: add missing deprecation number - #35630

Closed
bcoe wants to merge 1 commit into
nodejs:masterfrom
bcoe:assign-deprecation-number
Closed

doc: add missing deprecation number#35630
bcoe wants to merge 1 commit into
nodejs:masterfrom
bcoe:assign-deprecation-number

Conversation

bcoe commented Oct 13, 2020
edited
Loading

Copy link
Copy Markdown
Contributor

I assumed that DEP0XXX would be replaced automatically during release, like REPLACEME.

I was wrong. I bet this happens during git node land, and I have an old version of the bin.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

bcoe requested a review from Trott October 13, 2020 15:24
nodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Oct 13, 2020
bcoe added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 13, 2020

bcoe commented Oct 13, 2020

Copy link
Copy Markdown
Contributor Author

I would like to fast track this change, to unblock #35562 👍 👎

bcoe requested a review from richardlau October 13, 2020 15:28

Copy link
Copy Markdown
Collaborator

bcoe added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Oct 13, 2020

bcoe commented Oct 13, 2020

Copy link
Copy Markdown
Contributor Author

@BethGriggs we should make sure we land this in the LTS release too, I believe we're okay to back port the documentation only deprecation?

Copy link
Copy Markdown
Collaborator

bcoe added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 13, 2020
github-actions Bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 13, 2020

Copy link
Copy Markdown
Contributor

Landed in efe60a5...2cfdf28

github-actions Bot closed this Oct 13, 2020
nodejs-github-bot pushed a commit that referenced this pull request Oct 13, 2020
PR-URL: #35630
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
bcoe deleted the assign-deprecation-number branch October 13, 2020 18:45
MylesBorins pushed a commit that referenced this pull request Oct 14, 2020
PR-URL: #35630
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
MylesBorins mentioned this pull request Oct 14, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
PR-URL: nodejs#35630
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
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

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL