lint: eslint rule finding missing indentation after a comma when semicolon was expected

[Legends0]

Description

Unexpected indentation required flag for lines after a comma when semicolon was expected.

Additional Details / Steps to Reproduce

Saw this earlier today in JavaScript Lint / eslint (pull_request)

I'm unsure if this is expected behavior. The intention I had was that it should have been a semicolon.

Cactbot Module

Dev Tools / Internals

Configuration Info

Error is presented in GitHub checks.

Log & Screenshots

No response

Confirmation

• I reviewed Common Issues and Workarounds and verified that my settings are correct.
• I used the search function to confirm this issue doesn't already exist.

[valarnin]

It should have thrown a linting error on comma instead of semicolon, if it didn't that's an issue with the linter configuration.

[Bing-su]

However, cactbot's eslint 8 is already deprecated, and making any changes might require a major overhaul...

[Legends0]

It should have thrown a linting error on comma instead of semicolon, if it didn't that's an issue with the linter configuration.

It ignored the line 2299 data.blackHoleSet = data.blackHoleSet + 1, and interpreted:

        data.nothingnessCount = data.nothingnessCount + 1,
         data.blackHoleSet = data.blackHoleSet + 1;

But I think that fix seems less likely intended than replacing the comma.

[valarnin]

However, cactbot's eslint 8 is already deprecated, and making any changes might require a major overhaul...

The plan is to update a lot of cactbot's tech stack during the downtime between 7.56 (beastmaster release) and 8.0, though I might start on that early. The first part will be switching to vite which will necessitate updating eslint and a lot of other packages.