Commit e62c42b
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.
PR-URL: #6411
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 7153f96 commit e62c42b
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -7,7 +7,7 @@ | |||
| 7 | 7 | //------------------------------------------------------------------------------ | |
| 8 | 8 | // Rule Definition | |
| 9 | 9 | //------------------------------------------------------------------------------ | |
| 10 | - function getBinaryExpressionStarts(binaryExpression, starts = []) { | ||
| 10 | + function getBinaryExpressionStarts(binaryExpression, starts) { | ||
| 11 | 11 | function getStartsFromOneSide(side, starts) { | |
| 12 | 12 | starts.push(side.loc.start); | |
| 13 | 13 | if (side.type === 'BinaryExpression') { | |
@@ -29,7 +29,7 @@ function checkExpressionAlignment(expression) { | |||
| 29 | 29 | ||
| 30 | 30 | switch (expression.type) { | |
| 31 | 31 | case 'BinaryExpression': | |
| 32 | - var starts = getBinaryExpressionStarts(expression); | ||
| 32 | + var starts = getBinaryExpressionStarts(expression, []); | ||
| 33 | 33 | var startLine = starts[0].line; | |
| 34 | 34 | const startColumn = starts[0].column; | |
| 35 | 35 | starts.forEach((loc) => { | |
0 commit comments