Skip to content

doc: add missing deprecation code - #29969

Closed
cjihrig wants to merge 1 commit into
nodejs:masterfrom
cjihrig:dep
Closed

doc: add missing deprecation code#29969
cjihrig wants to merge 1 commit into
nodejs:masterfrom
cjihrig:dep

Conversation

@cjihrig

Copy link
Copy Markdown
Contributor

Refs: #29061

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

@nodejs-github-botnodejs-github-bot added the fs Issues and PRs related to the fs subsystem / file system. label Oct 14, 2019
@Trott

Copy link
Copy Markdown
Member

You can always count on me to mess this up.

@Trott

Copy link
Copy Markdown
Member

Collaborators, +1 here to fast-track. I'd like to make sure a release candidate doesn't go out without this, for example.

@TrottTrott added fast-track PRs that do not need to wait for 48 hours to land. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Oct 14, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@Trott

Copy link
Copy Markdown
Member

Getting a clean CI on this will probably require landing #29979 first.

@TrottTrott mentioned this pull request Oct 15, 2019
2 tasks
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@Trott

Copy link
Copy Markdown
Member

Landed in ff5a1cf

@TrottTrott closed this Oct 16, 2019
Trott pushed a commit that referenced this pull request Oct 16, 2019
Refs: #29061
PR-URL: #29969
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
@cjihrig
cjihrig deleted the dep branch October 16, 2019 13:50
BethGriggs pushed a commit that referenced this pull request Oct 22, 2019
Refs: #29061
PR-URL: #29969
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
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.fast-trackPRs that do not need to wait for 48 hours to land.fsIssues and PRs related to the fs subsystem / file system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@cjihrig@Trott@nodejs-github-bot@jasnell@Fishrock123@antsmartian@lpinca@devnexen@richardlau@targos