Skip to content

Conversation

@Trott
Copy link
Member

This includes better link linting, improved prohibited string linting,
and a more maintainable code base by using remark-preset-int-recommended
(thanks @nschonni!).

Update other dependencies while at it.

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 15, 2019
@Trott
Copy link
MemberAuthor

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 15, 2019
@Trott
Copy link
MemberAuthor

I don't want lint violations to land in the docs while this waits for the 48 hours, so I'd like to fast track. Collaborators, please 👍 here to approve.

@TrottTrott added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 15, 2019
Update vm.md for link linting rules that are about to be applied. Define an undefined link reference. Escape non-link text that uses `[` and `]`. PR-URL: nodejs#29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
This includes better link linting, improved prohibited string linting, and a more maintainable code base by using remark-preset-int-recommended (thanks nschonni!). Update other dependencies while at it. PR-URL: nodejs#29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@TrottTrottforce-pushed the update-preset-to-1.10.1 branch from cfccf45 to 90b5f1bCompareOctober 15, 2019 20:26
@Trott
Copy link
MemberAuthor

Landed in f4f856b...90b5f1b

@targos
Copy link
Member

Depends on #29864 to land on v12.x.

aduh95 pushed a commit to aduh95/node that referenced this pull request Jan 7, 2020
Update vm.md for link linting rules that are about to be applied. Define an undefined link reference. Escape non-link text that uses `[` and `]`. PR-URL: nodejs#29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
aduh95 pushed a commit to aduh95/node that referenced this pull request Jan 7, 2020
This includes better link linting, improved prohibited string linting, and a more maintainable code base by using remark-preset-int-recommended (thanks nschonni!). Update other dependencies while at it. PR-URL: nodejs#29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@aduh95aduh95 mentioned this pull request Jan 7, 2020
2 tasks
aduh95 pushed a commit to aduh95/node that referenced this pull request Jan 8, 2020
Update vm.md for link linting rules that are about to be applied. Define an undefined link reference. Escape non-link text that uses `[` and `]`. PR-URL: nodejs#29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
aduh95 pushed a commit to aduh95/node that referenced this pull request Jan 8, 2020
This includes better link linting, improved prohibited string linting, and a more maintainable code base by using remark-preset-int-recommended (thanks nschonni!). Update other dependencies while at it. PR-URL: nodejs#29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this pull request Jan 8, 2020
Update vm.md for link linting rules that are about to be applied. Define an undefined link reference. Escape non-link text that uses `[` and `]`. PR-URL: #29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this pull request Jan 8, 2020
This includes better link linting, improved prohibited string linting, and a more maintainable code base by using remark-preset-int-recommended (thanks nschonni!). Update other dependencies while at it. PR-URL: #29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@targostargos mentioned this pull request Jan 15, 2020
BethGriggs pushed a commit that referenced this pull request Feb 6, 2020
Update vm.md for link linting rules that are about to be applied. Define an undefined link reference. Escape non-link text that uses `[` and `]`. PR-URL: #29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
BethGriggs pushed a commit that referenced this pull request Feb 6, 2020
This includes better link linting, improved prohibited string linting, and a more maintainable code base by using remark-preset-int-recommended (thanks nschonni!). Update other dependencies while at it. PR-URL: #29982 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Feb 8, 2020
@TrottTrott deleted the update-preset-to-1.10.1 branch January 13, 2022 22:52
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.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@Trott@targos@lpinca@cjihrig@richardlau@BridgeAR@nodejs-github-bot