Skip to content

Commit e62c42b

Browse files
TrottFishrock123
authored andcommitted
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 <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 7153f96 commit e62c42b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tools/eslint-rules/align-multiline-assignment.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
//------------------------------------------------------------------------------
88
// Rule Definition
99
//------------------------------------------------------------------------------
10-
function getBinaryExpressionStarts(binaryExpression, starts = []) {
10+
function getBinaryExpressionStarts(binaryExpression, starts) {
1111
function getStartsFromOneSide(side, starts) {
1212
starts.push(side.loc.start);
1313
if (side.type === 'BinaryExpression') {
@@ -29,7 +29,7 @@ function checkExpressionAlignment(expression) {
2929

3030
switch (expression.type) {
3131
case 'BinaryExpression':
32-
var starts = getBinaryExpressionStarts(expression);
32+
var starts = getBinaryExpressionStarts(expression, []);
3333
var startLine = starts[0].line;
3434
const startColumn = starts[0].column;
3535
starts.forEach((loc) => {

0 commit comments

Comments
 (0)