Skip to content

tools: bump remark-lint-preset-node to 1.17.1 - #35668

Merged
Trott merged 0 commit into
nodejs:masterfrom
Trott:lint-md-update
Oct 18, 2020
Merged

tools: bump remark-lint-preset-node to 1.17.1#35668
Trott merged 0 commit into
nodejs:masterfrom
Trott:lint-md-update

Conversation

@Trott

Copy link
Copy Markdown
Member

First commit:

doc: correct order of metadata for deprecation
This is in preparation for updating remark-preset-lint-node to 1.17.1.

Second commit:

tools: bump remark-lint-preset-node to 1.17.1
Bump remark-lint-preset-node@1.16.0 to remark-lint-preset-node@1.17.1.
Refs: https://github.com/nodejs/remark-preset-lint-node/pull/139
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the tools Issues and PRs related to the tools directory. label Oct 16, 2020
@Trott
Trott requested review from aduh95 and bcoeOctober 16, 2020 04:20

@bcoebcoe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was the order of deprecation comments one of the things it lints?

This will make the deprecation process much easier 👍

@Trott

Copy link
Copy Markdown
MemberAuthor

Was the order of deprecation comments one of the things it lints?

Yes. This includes nodejs/remark-preset-lint-node#139 by @aduh95.

@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in ee85eb9...fbe33aa

Trott added a commit to Trott/io.js that referenced this pull request Oct 18, 2020
This is in preparation for updating remark-preset-lint-node to 1.17.1.
PR-URL: nodejs#35668
Reviewed-By: Ben Coe <bencoe@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@TrottTrott closed this Oct 18, 2020
@Trott
Trott merged commit fbe33aa into nodejs:masterOct 18, 2020
@Trott
Trott deleted the lint-md-update branch October 18, 2020 11:58
targos pushed a commit that referenced this pull request May 1, 2021
Bump remark-lint-preset-node@1.16.0 to remark-lint-preset-node@1.17.1.
Refs: nodejs/remark-preset-lint-node#139
PR-URL: #35668
Reviewed-By: Ben Coe <bencoe@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
danielleadams pushed a commit that referenced this pull request May 8, 2021
Bump remark-lint-preset-node@1.16.0 to remark-lint-preset-node@1.17.1.
Refs: nodejs/remark-preset-lint-node#139
PR-URL: #35668
Reviewed-By: Ben Coe <bencoe@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@bcoe@aduh95@targos@BethGriggs@nodejs-github-bot