Skip to content

Commit bd09cd5

Browse files
not-an-aardvarkkaicataldo
authored andcommitted
Update: avoid requiring NaN spaces of indentation (fixes #9083) (#9085)
This fixes an issue where the `indent` rule would sometimes expect "NaN spaces" of indentation when an `"off"` option was used.
1 parent c93a853 commit bd09cd5

File tree

2 files changed

+19
-1
lines changed

2 files changed

+19
-1
lines changed

lib/rules/indent.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -783,7 +783,7 @@ module.exports = {
783783
offsets.setDesiredOffsets(
784784
[startToken.range[1], endToken.range[0]],
785785
startToken,
786-
offset === "first" ? 1 : offset
786+
typeof offset === "number" ? offset : 1
787787
);
788788
offsets.setDesiredOffset(endToken, startToken, 0);
789789

tests/lib/rules/indent.js

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4915,6 +4915,24 @@ ruleTester.run("indent", rule, {
49154915
);
49164916
}
49174917
`
4918+
},
4919+
{
4920+
code: unIndent`
4921+
a(b
4922+
, c
4923+
)
4924+
`,
4925+
options: [2, { CallExpression: { arguments: "off" } }]
4926+
},
4927+
{
4928+
code: unIndent`
4929+
a(
4930+
new B({
4931+
c,
4932+
})
4933+
);
4934+
`,
4935+
options: [2, { CallExpression: { arguments: "off" } }]
49184936
}
49194937
],
49204938

0 commit comments

Comments
 (0)