actual icon indicating copy to clipboard operation
actual copied to clipboard

[Maintenance] Delete unused variables

Open MatissJanis opened this issue 2 years ago • 2 comments

We currently use @typescript-eslint/no-unused-vars eslint rule which is enabled for most things. However, it is disabled for args.

The overall goal here would be to re-enable @typescript-eslint/no-unused-vars for args. Thus disallowing unuser arguments.

https://github.com/actualbudget/actual/blob/08c80b6f585eaa3237100972f0edc06e45e444d1/.eslintrc.js#L54

Please do not do this as one-big-bang migration.

MatissJanis avatar Nov 14 '23 19:11 MatissJanis

I'm working on it

Sahad-09 avatar Nov 15 '23 11:11 Sahad-09

Sorry, didn't saw earlier that @Sahad-09 was working on this. Feel free to close my PR in favor of his.

jaas666 avatar Nov 18 '23 02:11 jaas666