doc: make markdown input compliant #21780
Conversation
|
We have some md lint issues: https://travis-ci.com/nodejs/node/jobs/134224344 |
|
@rubys FYI, on non-Windows, you can install the markdown linter with |
|
(And if you're on Windows, it should work with |
doc/api/stream.md
Outdated
| </table> | ||
| | Use-case | Class | Method(s) to implement | | ||
| | -------- | ----- | ---------------------- | | ||
| | Reading only |[`Readable`](#stream_class_stream_readable) |[_read][stream-_read]</code> | |
There was a problem hiding this comment.
The </code> part should likely have a beginning?
doc/api/stream.md
Outdated
| | Use-case | Class | Method(s) to implement | | ||
| | -------- | ----- | ---------------------- | | ||
| | Reading only |[`Readable`](#stream_class_stream_readable) |[_read][stream-_read]</code> | | ||
| |Writing only |[`Writable`](#stream_class_stream_writable) | <code>[_write][stream-_write]</code>, <code>[_writev][stream-_writev]</code>, <code>[_final][stream-_final]</code> | |
There was a problem hiding this comment.
Nit: a whitespace at the beginning of each part would be nice.
BridgeAR
added
the
author ready
| * `false`: causes a new `Agent` with default values to be used. | ||
| * `undefined` (default): use [`http.globalAgent`][] for this host and port. | ||
| * `Agent` object: explicitly use the passed in `Agent`. | ||
| * `false`: causes a new `Agent` with default values to be used. |
There was a problem hiding this comment.
Can this (proper padding) be enforced by remark-lint?
doc/api/stream.md
Outdated
| </table> | ||
| | Use-case | Class | Method(s) to implement | | ||
| | -------- | ----- | ---------------------- | | ||
| | Reading only | [`Readable`](#stream_class_stream_readable) | <code>[_read][stream-_read]</code> | |
There was a problem hiding this comment.
Why not [`Readable`][], [`Writable`][], [`Duplex`][], [`Transform`][]? That should make lines shorter.
Side note: there could be a test for that, i.e. to avoid redeclaring already declared refs.
|
Node.js Collaborators, please, add 👍 here if you approve fast-tracking. |
PR-URL: #21780 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
|
Landed in 6a99e3e |
PR-URL: #21780 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes