no-unused-variable doesn't properly handle "declare" vars

[derekcicerone-zz]

For example, this should not be flagged as unused:

declare var assert: chai.Assert;

[filipstachura]

+1

[marekrogala]

+1

[filipstachura]

any updates here?

[ashwinr]

we'll take a look at it as part of the 1.5 migration, should be straightforward.

[adidahiya]

It feels a little weird to me to not use a declared variable; I could see a case for this being an optional part of the no-unused-variable rule.

Is this because you're declaring a variable in one source file and using it in another? Or are you running tslint on your typings files? In the latter case, we'd advise only running tslint on source code.

[filipstachura]

We declare in one file and then reference it in others. That way when we refactor the name of injected global compilation fails in all files that were not updated.