1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

tools: remove unused trailing function arguments

Browse Source 
Update tools/doc/html.js and tools/eslint-rules/crypto-check.js to
remove unused trailing function arguments in preparation for enabling a
lint rule to enforce that practice.

PR-URL: https://github.com/nodejs/node/pull/16953
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@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>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
This commit is contained in:
Rich Trott 2017-11-09 14:27:12 +00:00 committed by James M Snell
parent dbb3c00d9d
commit bf2b5e1d5c
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/doc/html.js
View File

@ -224,7 +224,7 @@ function altDocs(filename) {
const host = 'https://nodejs.org'; const host = 'https://nodejs.org';
const href = (v) => `${host}/docs/latest-v${v.num}/api/${filename}.html`; const href = (v) => `${host}/docs/latest-v${v.num}/api/${filename}.html`;
function li(v, i) { function li(v) {
let html = `<li><a href="${href(v)}">${v.num}`; let html = `<li><a href="${href(v)}">${v.num}`;
if (v.lts) if (v.lts)

2
tools/eslint-rules/crypto-check.js
View File

@ -54,7 +54,7 @@ module.exports = function(context) {
} }
} }
function reportIfMissingCheck(node) { function reportIfMissingCheck() {
if (hasSkipCall) { if (hasSkipCall) {
return; return;
} }

2
tools/eslint-rules/inspector-check.js
View File

@ -29,7 +29,7 @@ module.exports = function(context) {
} }
} }
function reportIfMissing(context, node) { function reportIfMissing(context) {
if (!hasInspectorCheck) { if (!hasInspectorCheck) {
missingCheckNodes.forEach((node) => { missingCheckNodes.forEach((node) => {
context.report(node, msg); context.report(node, msg);