Browse Source

doc: use prefer-rest-params eslint rule in docs

Do not promote using of `arguments`.

One fragment is left as is because of history nature:
it uses a real deprecated code from libs.

Refs: http://eslint.org/docs/rules/prefer-rest-params
Refs: 99da8e8e02/lib/util.js (L1002-L1006)

PR-URL: https://github.com/nodejs/node/pull/13389
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
v6
Vse Mozhet Byt 8 years ago
parent
commit
ce8bce497c
  1. 1
      doc/.eslintrc.yaml
  2. 4
      doc/api/async_hooks.md
  3. 1
      doc/api/util.md

1
doc/.eslintrc.yaml

@ -12,3 +12,4 @@ rules:
# add new ECMAScript features gradually # add new ECMAScript features gradually
no-var: 2 no-var: 2
prefer-const: 2 prefer-const: 2
prefer-rest-params: 2

4
doc/api/async_hooks.md

@ -119,9 +119,9 @@ not invoke AsyncHooks recursively because it is synchronous.
const fs = require('fs'); const fs = require('fs');
const util = require('util'); const util = require('util');
function debug() { function debug(...args) {
// use a function like this one when debugging inside an AsyncHooks callback // use a function like this one when debugging inside an AsyncHooks callback
fs.writeSync(1, util.format.apply(null, arguments)); fs.writeSync(1, `${util.format(...args)}\n`);
} }
``` ```

1
doc/api/util.md

@ -55,6 +55,7 @@ added: v0.8.0
The `util.deprecate()` method wraps the given `function` or class in such a way that The `util.deprecate()` method wraps the given `function` or class in such a way that
it is marked as deprecated. it is marked as deprecated.
<!-- eslint-disable prefer-rest-params -->
```js ```js
const util = require('util'); const util = require('util');

Loading…
Cancel
Save