Skip to content

doc: nest code fence under unordered list - #29915

Closed
nschonni wants to merge 2 commits into
nodejs:masterfrom
nschonni:fix-cve-management
Closed

doc: nest code fence under unordered list#29915
nschonni wants to merge 2 commits into
nodejs:masterfrom
nschonni:fix-cve-management

Conversation

@nschonni

Copy link
Copy Markdown
Member
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

When it isn't nested, a new unordered list is created for the next item
@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 10, 2019
@BridgeAR

Copy link
Copy Markdown
Member

@nschonni thank you for looking closely into the documentation formatting!

@nschonni

Copy link
Copy Markdown
MemberAuthor

No problem, just see what the gaps are between the Remark-lint preset and Markdownlint :)

@Trott

Copy link
Copy Markdown
Member

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 10, 2019
Trott pushed a commit that referenced this pull request Oct 13, 2019
PR-URL: #29915
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Trott pushed a commit that referenced this pull request Oct 13, 2019
When it isn't nested, a new unordered list is created for the next item
PR-URL: #29915
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@Trott

Copy link
Copy Markdown
Member

Landed in 19a8d22...40ef537

@TrottTrott closed this Oct 13, 2019
@nschonni
nschonni deleted the fix-cve-management branch October 13, 2019 00:35
targos pushed a commit that referenced this pull request Oct 14, 2019
PR-URL: #29915
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Oct 14, 2019
When it isn't nested, a new unordered list is created for the next item
PR-URL: #29915
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@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.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@nschonni@BridgeAR@Trott@lpinca@cjihrig@trivikr@nodejs-github-bot