add typechecking to tslint

[GabrielCastro]

This solves #126,
I ran eject on a project made these changes and no longer get the flash error from tslint

[gsteacy]

I'm not sure type checking should be enabled while this is still an issue: wbuchwalter/tslint-loader#76. Even on small projects I noticed a performance hit of 250-300%.

[pelotom]

@gsteacy agreed, I've stopped using tslint-loader and switched to tslint-webpack-plugin because of this.

[stevenmusumeche]

@pelotom are you able to do that without ejecting though?

[pelotom]

@stevenmusumeche no.

[wmonk]

As this issue hasn't been updated for a while, I am going to close it. If there are any updates feel free to re-open and comment. 👍

[DorianGrey]

FYI: fork-ts-checker-webpack-plugin, which was added in 2.10, supports linting rules that require type checking by sharing ts.Program instances, so that feature should be available anyway.

[pelotom]

Yes, however be aware that fork-ts-checker-webpack-plugin has a bug with the no-unused-variable rule which may or may not bite you. For my project it was intolerable; I had to disable the rule and use the tsc —noUnusedLocals flag instead.

[DorianGrey]

That's currently the case in the template - noUnusedLocals is used as a compiler option, no-unused-variable is not part of the linter config.