linter: update eslint flag

[molecula451]

Resolves #810

QA:

[ubiquibot]

• 41e53b8

[rndquu]

I don't understand the purpose of this code change. I tried both with and without the "!" symbol in the cspell.json file and it doesn't affect the build.

We already have the non-null assertion rule enabled hence the warnings are shown. What we should do is to throw an error if there are some warnings produced by eslint so that the CI build failed with error. This line could be updated to eslint --fix --max-warnings=0 (more info)

[molecula451]

lmao, well the thing is i rarely use ! in the code, and always fix my code to eslint errors i thought that was why the warning pop by updating the cspell.json will have to recheck that a bit

so basically the contributor ignored the warning, we should have updated to a fail/error instead

[molecula451]

i think --max-warnings=0 is approachable so we keep it simple and fix errors when eslint pop em up