-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Enable Embroider by default for new projects #9495
Conversation
Fixes: embroider-build#681 & embroider-build#676 Ember-cli's test suite caught an issue with windows support when trying to enable embroider by default (ember-cli/ember-cli#9495). After this change I was able to successfully do a build with a new Ember app.
Will need to have: embroider-build/embroider#742 landed and released |
Fixes: embroider-build#681 & embroider-build#676 Ember-cli's test suite caught an issue with windows support when trying to enable embroider by default (ember-cli/ember-cli#9495). After this change I was able to successfully do a build with a new Ember app.
Fixes: embroider-build#681 & embroider-build#676 Ember-cli's test suite caught an issue with windows support when trying to enable embroider by default (ember-cli/ember-cli#9495). After this change I was able to successfully do a build with a new Ember app.
c660901
to
4fc54d5
Compare
@rwjblue this is good to go now |
🎉🥁🙌👏 |
"@embroider/compat": "^0.37.0", | ||
"@embroider/core": "^0.37.0", | ||
"@embroider/webpack": "^0.37.0<% } %>", | ||
"@embroider/compat": "^0.39.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be 1.0 if we are now official?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not necessarily (we have other non-1.0.0 things in the blueprint, e.g. ember-maybe-import-regenerator
), but we absolutely are working towards a 1.0.0. Look out for a meta issue on Embroider repo soon for that. I'd like to get that landed by the time we do 3.28.0 (an LTS release).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cross linking the issue you made to track that also: embroider-build/embroider#761
As much as I am excited about Embroider (having used and contributed to it), I am nevertheless quite surprised by this move here! Given how much this changes the underlying build infrastructure, wouldn't an RFC be required here? Mainly for two reasons:
Some examples for these points off the top of my head:
Tl;dr: I feel this is premature and would require a thorough transition path in an RFC! 🤷♂️ /cc @ef4 |
New apps generated via
ember new
will now default to use Embroider's build system.