search

microsoft / vscode-typescript-tslint-plugin

VS Code extension that provides TSLint support using the typescript-tslint-plugin
https://marketplace.visualstudio.com/items?itemName=ms-vscode.vscode-typescript-tslint-plugin
MIT License
188 stars 35 forks source link

'no-unused-variable' issues are not being linted #71

Closed evenfrost closed 5 years ago

evenfrost commented 5 years ago

In short: what is the status of implementing the linting of this rule in current VS Code TSLint plugin? Old version couldn't handle it, what about this one?

I know there's been plenty of discussion about TypeScript's noUnusedLocals vs TSLint's no-unused-variable, and latter being deprecated first but now due to user response it's considered to be undeprecated. The main argument in favour of leaving no-unused-variable as it is that it helps in development process not throwing errors on compiling (as noUnusedLocals does) but being worked out later as you commit (using pre-commit linting e.g. with husky and lint-staged).

mjbvz commented 5 years ago

No plans to do extra work to support this