Skip to content

Commit 19ca6cd

Browse files
sam-githubaddaleax
authored andcommitted
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 <[email protected]> Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]>
1 parent 7c1a2f5 commit 19ca6cd

File tree

6 files changed

+8
-11
lines changed

6 files changed

+8
-11
lines changed

.editorconfig

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,6 @@ insert_final_newline = true
99
[vcbuild.bat]
1010
end_of_line = crlf
1111

12-
[*.{md,markdown}]
13-
trim_trailing_whitespace = false
14-
1512
[{lib,src,test}/**.js]
1613
indent_style = space
1714
indent_size = 2

BUILDING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ Prerequisites:
2020

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

benchmark/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,7 @@ const bench = common.createBenchmark(main, {
326326
function main(conf) {
327327
const http = require('http');
328328
const len = conf.kb * 1024;
329-
const chunk = Buffer.alloc(len, 'x');
329+
const chunk = Buffer.alloc(len, 'x');
330330
const server = http.createServer(function(req, res) {
331331
res.end(chunk);
332332
});

doc/changelogs/CHANGELOG_V010.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@
6868
* [io.js](CHANGELOG_IOJS.md)
6969
* [Archive](CHANGELOG_ARCHIVE.md)
7070

71-
**Note:** Node.js v0.10 is covered by the
71+
**Note:** Node.js v0.10 is covered by the
7272
[Node.js Long Term Support Plan](https://github.com/nodejs/LTS) and
7373
will be maintained until October 2016.
7474

@@ -311,11 +311,11 @@ https://github.com/nodejs/node/commit/8d045a30e95602b443eb259a5021d33feb4df079
311311
* child_process: properly support optional args (cjihrig)
312312
* crypto: Disable autonegotiation for SSLv2/3 by default (Fedor Indutny,
313313
Timothy J Fontaine, Alexis Campailla)
314-
314+
315315
This is a behavior change, by default we will not allow the negotiation to
316316
SSLv2 or SSLv3. If you want this behavior, run Node.js with either
317317
`--enable-ssl2` or `--enable-ssl3` respectively.
318-
318+
319319
This does not change the behavior for users specifically requesting
320320
`SSLv2_method` or `SSLv3_method`. While this behavior is not advised, it is
321321
assumed you know what you're doing since you're specifically asking to use

doc/changelogs/CHANGELOG_V012.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
* [io.js](CHANGELOG_IOJS.md)
3636
* [Archive](CHANGELOG_ARCHIVE.md)
3737

38-
**Note:** Node.js v0.12 is covered by the
38+
**Note:** Node.js v0.12 is covered by the
3939
[Node.js Long Term Support Plan](https://github.com/nodejs/LTS) and
4040
will be maintained until December 31st, 2016.
4141

test/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ and `setInterval`).
146146
## Common module API
147147

148148
The common.js module is used by tests for consistency across repeated
149-
tasks. It has a number of helpful functions and properties to help with
149+
tasks. It has a number of helpful functions and properties to help with
150150
writing tests.
151151

152152
### allowGlobals(...whitelist)
@@ -177,7 +177,7 @@ Check if there is more than 1gb of total memory.
177177
* `name` [&lt;String>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#String_type)
178178
* `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)
179179

180-
Tests whether `name` and `expected` are part of a raised warning.
180+
Tests whether `name` and `expected` are part of a raised warning.
181181

182182
### hasCrypto
183183
* return [&lt;Boolean>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#Boolean_type)

0 commit comments

Comments
 (0)