TSLint #68
TSLint #68
Conversation
| import { ComponentManager, setPropertyDidChange } from '@glimmer/component'; | ||
| import App from './main'; |
There was a problem hiding this comment.
Which rule is forcing this change? I don’t fully understand why the various import statements are moved around...
There was a problem hiding this comment.
This change is caused by ordered-imports which is included in the recommended preset.
There was a problem hiding this comment.
@rwjblue do you prefer to disable the rule or keep the changes? I'd go with the changes.
There was a problem hiding this comment.
I’m not a giant fan of this specific rule, but meh.
There was a problem hiding this comment.
Sadly the recommended preset comes with too much opinions about stylistic choices in my opinion. I prefer what ESLint is doing in their recommended preset (not including any stylistic choices at all) but as far as I know, no similar preset exists for TSLint.
Added
ember-cli-tslintto the blueprint and adjusted the code/config to pass the tests after initialisation.