Remove Deprecated no-unused-variable Rule

[todd]

TSLint has deprecated no-unused-variable and now recommends using the built in compiler checks in TypeScript >= 2.9 to achieve the same result.

[krzkaczor]

Hey,
the problem that I have with this setting is that it interrupts compiling. Also I can't find any note about deprecation in docs :/ https://palantir.github.io/tslint/rules/no-unused-variable/ Am i missing something?

[todd]

See palantir/tslint#3919 for details on the deprecation - I guess they haven't updated their docs yet. However, TSLint will log a deprecation warning when attempting to use this option, which is how I found out about it in the first place.

[krzkaczor]

Yeah, I noticed that warning as well but given the current state of tsc flag, i can't agree to remove this rule.

Here's a little bit more discussion about removing deprecation: palantir/tslint#4100 (also it was de-deprecated already once xD) main points being:

• it's compiler error not warning
• you cannot specify ignore pattern

Thanks for bringing this issue and I am sure that we should reconsider this in a few months and see where the community goes. I will leave a note in the readme about this issue.