-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make ember-cli-typescript-blueprints optional #367
Comments
Thanks for this suggestion! It’s a good call. If you feel like opening a PR, we’d be really happy to review and work with you to get it merged! Otherwise we’ll see about getting to it... at some point. |
Just to clarify the current status quo:
|
The status quo is the correct behavior in v2. Consumers may want to "opt out" but they should get TS blueprints by default. We'd also probably end up convincing @championswimmer to keep them around after our work on #364 - where we fully embrace decorators as part of the ember-cli-typescript experience. imo the blueprints would end up being exactly what he wants. I don't think any action is required here, unless we want to consder back-porting something to the v1 release series |
Ok since 2.0.0-x is still in beta, I didn't notice the new behaviour in 2.0. It is completely acceptable to me what 2.0 does. (Probably adding in the documentation that removing it simply from package.json is enough should be good) |
Awesome, glad to hear that solves it well for you – and no worries about not knowing just exactly what things are coming in v2.0! 😆 |
Please write a user story for this feature
There are cases like this
ember-decorators
and do not want to useextends Object.extend({})
syntax and simple useextends Object {}
syntaxWhat I would prefer is that I can use the older js blueprints or the
ember-decorators
blueprints and not use ts blueprints.So when installing if
ember-cli-typescript
asks me if I wantember-cli-typescript-blueprints
instead of always installing, it would be better.The text was updated successfully, but these errors were encountered: