eslint warnings

[dadofsambonzuki]

There are hundreds of eslint warnings throughout the project.

A thankless task, but some hero should address them.

[escapedcat]

Status update (after #240 is merged):

✖ 213 problems (0 errors, 213 warnings)

These are coming from the left over rules. A good start would be to tackle the rule 'svelte/require-each-key': 'warn',

Steps:

• comment rule out here
• run yarn lint
• fix the svelte loops one by one while testing if each view/component is still working as expected

Let us know if you need any help or you have any questions regarding this task

[kwilson62]

Anyone working this yet? If not, I can start working through them as I learn the code base.

[escapedcat]

@kwilson62 go ahead, let us know if you get stuck

[escapedcat]

Update after #257

Before:

✖ 194 problems (0 errors, 194 warnings)

After:

✖ 127 problems (0 errors, 127 warnings)

[escapedcat]

Example on how to fix (some?) @typescript-eslint/no-unused-expressions warnings.

Via: https://github.com/sveltejs/eslint-plugin-svelte/issues/934#issuecomment-2543128219

[escapedcat]

Further fixing examples

(AI helped)