glimmerjs / glimmer-application

[MOVED] This package is now part of the Glimmer.js monorepo
https://github.com/glimmerjs/glimmer.js
30 stars 13 forks source link

Added TSLint #59

Closed t-sauer closed 7 years ago

t-sauer commented 7 years ago

I started with the recommended preset and just changed rules to match the current style or which would lead to way too complex changes (e.g. max-classes-per-file).
If you want any other rules I would be glad to change them.

t-sauer commented 7 years ago

I disabled the prefer-const rule.

dgeb commented 7 years ago

@t-sauer Thanks for your work here.

We have a few things to discuss and decide upon as a team:

We'll try to get back to you soon to minimize churn.

t-sauer commented 7 years ago

Seems like TSLint was already added so this PR isn't needed anymore.