tools: disallow trailing whitespace for markdown

markdown had a dispensation because 2 or more trailing spaces triggers a
new paragraph. There are no examples of that usage in Node, all trailing
whitespace found were mistakes, and the dispensation is now removed.

See: https://github.com/nodejs/node/pull/9620
PR-URL: https://github.com/nodejs/node/pull/9676
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
This commit is contained in:
Sam Roberts 2016-11-18 09:40:45 -08:00
parent 13c4f44e97
commit 3cbb553df8
6 changed files with 8 additions and 11 deletions

View File

@ -9,9 +9,6 @@ insert_final_newline = true
[vcbuild.bat] [vcbuild.bat]
end_of_line = crlf end_of_line = crlf
[*.{md,markdown}]
trim_trailing_whitespace = false
[{lib,src,test}/**.js] [{lib,src,test}/**.js]
indent_style = space indent_style = space
indent_size = 2 indent_size = 2

View File

@ -20,7 +20,7 @@ Prerequisites:
On OS X, you will also need: On OS X, you will also need:
* [Xcode](https://developer.apple.com/xcode/download/) * [Xcode](https://developer.apple.com/xcode/download/)
* You also need to install the `Command Line Tools` via Xcode. You can find * You also need to install the `Command Line Tools` via Xcode. You can find
this under the menu `Xcode -> Preferences -> Downloads` this under the menu `Xcode -> Preferences -> Downloads`
* This step will install `gcc` and the related toolchain containing `make` * This step will install `gcc` and the related toolchain containing `make`

View File

@ -326,7 +326,7 @@ const bench = common.createBenchmark(main, {
function main(conf) { function main(conf) {
const http = require('http'); const http = require('http');
const len = conf.kb * 1024; const len = conf.kb * 1024;
const chunk = Buffer.alloc(len, 'x'); const chunk = Buffer.alloc(len, 'x');
const server = http.createServer(function(req, res) { const server = http.createServer(function(req, res) {
res.end(chunk); res.end(chunk);
}); });

View File

@ -70,7 +70,7 @@
* [io.js](CHANGELOG_IOJS.md) * [io.js](CHANGELOG_IOJS.md)
* [Archive](CHANGELOG_ARCHIVE.md) * [Archive](CHANGELOG_ARCHIVE.md)
**Note:** Node.js v0.10 is covered by the **Note:** Node.js v0.10 is covered by the
[Node.js Long Term Support Plan](https://github.com/nodejs/LTS) and [Node.js Long Term Support Plan](https://github.com/nodejs/LTS) and
will be maintained until October 2016. will be maintained until October 2016.
@ -359,11 +359,11 @@ https://github.com/nodejs/node/commit/8d045a30e95602b443eb259a5021d33feb4df079
* child_process: properly support optional args (cjihrig) * child_process: properly support optional args (cjihrig)
* crypto: Disable autonegotiation for SSLv2/3 by default (Fedor Indutny, * crypto: Disable autonegotiation for SSLv2/3 by default (Fedor Indutny,
Timothy J Fontaine, Alexis Campailla) Timothy J Fontaine, Alexis Campailla)
This is a behavior change, by default we will not allow the negotiation to This is a behavior change, by default we will not allow the negotiation to
SSLv2 or SSLv3. If you want this behavior, run Node.js with either SSLv2 or SSLv3. If you want this behavior, run Node.js with either
`--enable-ssl2` or `--enable-ssl3` respectively. `--enable-ssl2` or `--enable-ssl3` respectively.
This does not change the behavior for users specifically requesting This does not change the behavior for users specifically requesting
`SSLv2_method` or `SSLv3_method`. While this behavior is not advised, it is `SSLv2_method` or `SSLv3_method`. While this behavior is not advised, it is
assumed you know what you're doing since you're specifically asking to use assumed you know what you're doing since you're specifically asking to use

View File

@ -37,7 +37,7 @@
* [io.js](CHANGELOG_IOJS.md) * [io.js](CHANGELOG_IOJS.md)
* [Archive](CHANGELOG_ARCHIVE.md) * [Archive](CHANGELOG_ARCHIVE.md)
**Note:** Node.js v0.12 is covered by the **Note:** Node.js v0.12 is covered by the
[Node.js Long Term Support Plan](https://github.com/nodejs/LTS) and [Node.js Long Term Support Plan](https://github.com/nodejs/LTS) and
will be maintained until December 31st, 2016. will be maintained until December 31st, 2016.

View File

@ -146,7 +146,7 @@ and `setInterval`).
## Common module API ## Common module API
The common.js module is used by tests for consistency across repeated The common.js module is used by tests for consistency across repeated
tasks. It has a number of helpful functions and properties to help with tasks. It has a number of helpful functions and properties to help with
writing tests. writing tests.
### allowGlobals(...whitelist) ### allowGlobals(...whitelist)
@ -177,7 +177,7 @@ Check if there is more than 1gb of total memory.
* `name` [&lt;String>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#String_type) * `name` [&lt;String>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#String_type)
* `expected` [&lt;String>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#String_type) | [&lt;Array>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array) * `expected` [&lt;String>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#String_type) | [&lt;Array>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array)
Tests whether `name` and `expected` are part of a raised warning. Tests whether `name` and `expected` are part of a raised warning.
### hasCrypto ### hasCrypto
* return [&lt;Boolean>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#Boolean_type) * return [&lt;Boolean>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#Boolean_type)