Skip to content

doc: unify deprecation wording - #22555

Closed
tniessen wants to merge 2 commits into
nodejs:masterfrom
tniessen:doc-unify-deprecation-wording
Closed

doc: unify deprecation wording#22555
tniessen wants to merge 2 commits into
nodejs:masterfrom
tniessen:doc-unify-deprecation-wording

Conversation

@tniessen

Copy link
Copy Markdown
Member

As suggested by @Trott in #22519 (comment): This is the only occurrence of has been deprecated.

Checklist

@tniessen
tniessen requested a review from TrottAugust 27, 2018 17:43
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Aug 27, 2018
@vsemozhetbyt

Copy link
Copy Markdown
Contributor

Node.js Collaborators, please, add 👍 here if you approve fast-tracking.

@vsemozhetbytvsemozhetbyt added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Aug 27, 2018
Comment threaddoc/api/deprecations.md Outdated
Type: Documentation-only

The option `produceCachedData` has been deprecated. Use
The option `produceCachedData` is deprecated. Use

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.

Micro-nit, can be ignored: There are five instances of suchAndSuch option and this is the only instance of option suchAndSuch. Change this to produceCacheData option rather than option produceCacheData.

@tniessentniessen added fast-track PRs that do not need to wait for 48 hours to land. and removed fast-track PRs that do not need to wait for 48 hours to land. labels Aug 29, 2018
@tniessen

Copy link
Copy Markdown
MemberAuthor

Landed in f417ea1.

tniessen added a commit that referenced this pull request Aug 30, 2018
PR-URL: #22555
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
addaleax pushed a commit that referenced this pull request Aug 31, 2018
PR-URL: #22555
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Sep 3, 2018
PR-URL: #22555
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Sep 5, 2018
targos pushed a commit that referenced this pull request Sep 6, 2018
PR-URL: #22555
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@tniessentniessen removed the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 12, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deprecationsIssues and PRs related to deprecations.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@tniessen@nodejs-github-bot@vsemozhetbyt@jasnell@Trott@addaleax@lpinca@trivikr