Permalink
Browse files

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: #9620
PR-URL: #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>
  • Loading branch information...
sam-github committed Nov 18, 2016
1 parent 13c4f44 commit 3cbb553df8fefbed7fd09b4b75f1d6f7b611d3b5
Showing with 8 additions and 11 deletions.
  1. +0 −3 .editorconfig
  2. +1 −1 BUILDING.md
  3. +1 −1 benchmark/README.md
  4. +3 −3 doc/changelogs/CHANGELOG_V010.md
  5. +1 −1 doc/changelogs/CHANGELOG_V012.md
  6. +2 −2 test/README.md
@@ -9,9 +9,6 @@ insert_final_newline = true
[vcbuild.bat]
end_of_line = crlf
[*.{md,markdown}]
trim_trailing_whitespace = false
[{lib,src,test}/**.js]
indent_style = space
indent_size = 2
@@ -20,7 +20,7 @@ Prerequisites:
On OS X, you will also need:
* [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 step will install `gcc` and the related toolchain containing `make`
@@ -326,7 +326,7 @@ const bench = common.createBenchmark(main, {
function main(conf) {
const http = require('http');
const len = conf.kb * 1024;
const chunk = Buffer.alloc(len, 'x');
const chunk = Buffer.alloc(len, 'x');
const server = http.createServer(function(req, res) {
res.end(chunk);
});
@@ -70,7 +70,7 @@
* [io.js](CHANGELOG_IOJS.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
will be maintained until October 2016.
@@ -359,11 +359,11 @@ https://github.com/nodejs/node/commit/8d045a30e95602b443eb259a5021d33feb4df079
* child_process: properly support optional args (cjihrig)
* crypto: Disable autonegotiation for SSLv2/3 by default (Fedor Indutny,
Timothy J Fontaine, Alexis Campailla)
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
`--enable-ssl2` or `--enable-ssl3` respectively.
This does not change the behavior for users specifically requesting
`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
@@ -37,7 +37,7 @@
* [io.js](CHANGELOG_IOJS.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
will be maintained until December 31st, 2016.
@@ -146,7 +146,7 @@ and `setInterval`).
## Common module API
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.
### 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)
* `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
* return [&lt;Boolean>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#Boolean_type)

0 comments on commit 3cbb553

Please sign in to comment.