From ce8bce497c788a8d4788ff5da7d48cd98228e917 Mon Sep 17 00:00:00 2001 From: Vse Mozhet Byt Date: Fri, 2 Jun 2017 12:20:47 +0300 Subject: 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: https://github.com/nodejs/node/blob/99da8e8e02a874a0a044889f863c45700509d02c/lib/util.js#L1002-L1006 PR-URL: https://github.com/nodejs/node/pull/13389 Reviewed-By: Daijiro Wachi Reviewed-By: Colin Ihrig Reviewed-By: Luigi Pinca Reviewed-By: James M Snell Reviewed-By: Refael Ackermann --- doc/.eslintrc.yaml | 1 + doc/api/async_hooks.md | 4 ++-- doc/api/util.md | 1 + 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/doc/.eslintrc.yaml b/doc/.eslintrc.yaml index 52e5b7eef9..5b7ea75de5 100644 --- a/doc/.eslintrc.yaml +++ b/doc/.eslintrc.yaml @@ -12,3 +12,4 @@ rules: # add new ECMAScript features gradually no-var: 2 prefer-const: 2 + prefer-rest-params: 2 diff --git a/doc/api/async_hooks.md b/doc/api/async_hooks.md index 2cdc160cb9..c2a55a97d3 100644 --- a/doc/api/async_hooks.md +++ b/doc/api/async_hooks.md @@ -119,9 +119,9 @@ not invoke AsyncHooks recursively because it is synchronous. const fs = require('fs'); const util = require('util'); -function debug() { +function debug(...args) { // 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`); } ``` diff --git a/doc/api/util.md b/doc/api/util.md index ed0f2882e9..839bdadf76 100644 --- a/doc/api/util.md +++ b/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 it is marked as deprecated. + ```js const util = require('util'); -- cgit v1.2.3