Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34.3k
doc: fix deprecation "End-of-Life" capitalization#26251
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: fix deprecation "End-of-Life" capitalization #26251
Uh oh!
There was an error while loading. Please reload this page.
Conversation
nodejs-github-bot commented Feb 21, 2019
Trott left a comment • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Might be good to add End-Of-Life to https://github.com/Trott/remark-preset-lint-node/blob/594048a86e09993ead132df1c499fee05a239bce/index.js#L49-L55 to keep it from creeping back in.
cjihrig left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, and agree with @Trott's idea.
Trott commented Feb 21, 2019
Once this lands, I'm happy to do it, but a PR is of course welcome! |
cjihrig commented Feb 21, 2019
I thought it would make the most sense as part of this PR, but yea, it can be separate too. |
richardlau commented Feb 22, 2019
Since this was easy enough to do in GitHub's web UI editor: nodejs/remark-preset-lint-node#12 |
Trott commented Feb 23, 2019
Landed in 7e0ddf6 |
PR-URL: nodejs#26251 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
Update remark-preset-lint-node to 1.4.0. This adds `End-Of-Life` as a prohibited string, favoring `End-of-Life` for consistency. Refs: nodejs#26251
Update remark-preset-lint-node to 1.4.0. This adds `End-Of-Life` as a prohibited string, favoring `End-of-Life` for consistency. Refs: nodejs#26251 PR-URL: nodejs#26281 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
Update remark-preset-lint-node to 1.4.0. This adds `End-Of-Life` as a prohibited string, favoring `End-of-Life` for consistency. Refs: #26251 PR-URL: #26281 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
Update the "End-of-Life" casing in the deprecation documentation for consistency. Similar to nodejs#26251. PR-URL: nodejs#33691 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
There are 78 occurrences of
End-of-Lifeand two occurrences ofEnd-Of-Life.Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes