no-unused-variable is deprecated

[LinusU]

no-unused-variable is deprecated. Since TypeScript 2.9. Please use the built-in compiler checks instead.

Should we just disable the rule?

Personally, I'm not a fan of moving lint checks into the compiler, I think... Oh well 🤷‍♂️

[blakeembrey]

Me either. I think this was already fixed though, just needs to be released. Will follow up over the weekend.

[LinusU]

Yeah, looks like it was fixed here: 1a99e55

@blakeembrey : can you publish a new release?

@blakeembrey : hmm maybe it might be wise to support typescript 3.x too, but i'm not sure when tslint will publish their 3.x support though.

[blakeembrey]

Is this somewhat urgent for you? As far as I understand it’s a deprecation, is it breaking something?

If there were no other changes coming soon then I would want the release. In this case though, might as well wait for a typescript 3.x compatible tslint.

[boycce]

Any chance we publish a new release, linting deprecation notices are a pain in Sublime.

It'd be nice to see #53 added before a new release is cut.

[adrianhopebailie]

Minor changes are what npm patch releases are for.
A quick patch (v7.1.1) that includes 1a99e55 would be great

[blakeembrey]

This would be a major release, not a patch. It’s removing a feature.

[Pilatch]

I, too, am distracted by my linter yelling at me each time I open my text editor and point it at a TS file.

It burns my biscuits, but does not fry my loins. Therefore I will wait for 8.