-
Notifications
You must be signed in to change notification settings - Fork 4
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
Use local beta version of ember-cli #87
Conversation
Run & review this pull request in StackBlitz Codeflow. |
|
de7a8c0
to
8c8cd1b
Compare
cli/src/v2-blueprint.js
Outdated
info.name, | ||
`--dir=${addonDir}`, | ||
// needs https://github.com/embroider-build/addon-blueprint/pull/137 for proper TS support | ||
'--blueprint=embroider-build/addon-blueprint#update-for-upstream-ts-changes', |
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.
- Use a new published version once we have Update for ember-cli
--typescript
changes embroider-build/addon-blueprint#137 merged.
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.
Reverted that, it's now using the npm published blueprint again, which as of https://github.com/embroider-build/addon-blueprint/releases/tag/v1.7.0 contains all the things we need
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.
Yay!
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.
Love all these deletions 🎉
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.
What could go wrong? 🙃
Tho. CLI shourd be released in the next couple days 🤞
(I'm approving this, since i it fixes the ts stuff, but we'll want to at least back off to beta once available)
((I can always hold off on publishing to npm until real versions are used))
This changes the way we invoke ember-cli, to use our own dependency, so we control the version of the blueprint. And (for now) we use the latest canary version, to use the fixed `--typescript` support.
I rebased and changed to again use the npm published blueprint, but now tests are failing 🤔 ... need to investigate... |
@@ -44,7 +44,6 @@ | |||
"ember-cli-sri": "^2.1.1", | |||
"ember-cli-terser": "^4.0.2", | |||
"ember-disable-prototype-extensions": "^1.1.3", | |||
"ember-export-application-global": "^2.0.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.
Had to remove this, which is deprecated anyway, as with the new ember-cli we get Ember 5 apps created, which this is incompatible with. This was the reason for CI failing...
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.
I wonder if the migration code should remove it?
References 5.2.0-beta.0 now instead of master! :) |
This changes the way we invoke ember-cli, to use our own dependency, so we control the version of the blueprint. And (for now) we use the latest canary version, to use the fixed
--typescript
support.