-
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
Prepares Ember CLI for new version of ember-welcome-page #6378
Conversation
@ember-cli/core Thoughts on having Ember CLI run the default blueprint of any addon present after I'm hesitant to enshrine what feel like blueprint related tasks into our blueprints. If y'all feel like that's a good feature I'll build it today (so that welcome-page may adopt it). |
@nathanhammond 💯 👍 (see #6269) |
Sounds good to me. |
I'm less sure about |
We should have this as a different thing 🤔 |
What about the scenario, you're doing an ember-cli upgrade via Isn't that the same problem as with |
@kellyselden You make a good point. But unfortunately that implies that default blueprints must be idempotent/safe to rerun which is not something that we require at this time since they're only ever invoked once. This is a larger problem than I was thinking and I no longer want to block this PR on that effort. |
Good point as well, I wasn't thinking of the blowing away of previous custom changes to blueprint. Carry on, no changes to |
Hokay, so, @locks, this is not blocked by CLI people's opinions. Just tests passing once the new version is released. |
2e51b01
to
0b7721f
Compare
- Update ember-welcome-page dependency - Re-introduce application.hbs template to include the new welcome-page component
@homu r+ |
📌 Commit 58b7431 has been approved by |
@locks Thank you for your tireless efforts on this! |
Prepares Ember CLI for new version of ember-welcome-page - Update ember-welcome-page dependency - Re-introduce application.hbs template to include the new welcome-page component - [x] Release new `ember-welcome-page` version
☀️ Test successful - status |
welcome-page component
ember-welcome-page
version