From dab9658a86fc2b53319bd8800ce26141b875177f Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Tue, 26 Apr 2016 22:14:28 -0700 Subject: [PATCH] tools: remove default parameters from lint rule The CI server uses system Node.js for linting, which is currently v5.x. So default parameters are not supported there. This change removes the default parameters. --- tools/eslint-rules/align-multiline-assignment.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/eslint-rules/align-multiline-assignment.js b/tools/eslint-rules/align-multiline-assignment.js index 69f1f33d4d184f..80896b5e96296d 100644 --- a/tools/eslint-rules/align-multiline-assignment.js +++ b/tools/eslint-rules/align-multiline-assignment.js @@ -7,7 +7,7 @@ //------------------------------------------------------------------------------ // Rule Definition //------------------------------------------------------------------------------ -function getBinaryExpressionStarts(binaryExpression, starts = []) { +function getBinaryExpressionStarts(binaryExpression, starts) { function getStartsFromOneSide(side, starts) { starts.push(side.loc.start); if (side.type === 'BinaryExpression') { @@ -29,7 +29,7 @@ function checkExpressionAlignment(expression) { switch (expression.type) { case 'BinaryExpression': - var starts = getBinaryExpressionStarts(expression); + var starts = getBinaryExpressionStarts(expression, []); var startLine = starts[0].line; const startColumn = starts[0].column; starts.forEach((loc) => {