Skip to content

update the format of the ember-cli-update.json#10791

Merged
mansona merged 3 commits intomasterfrom
ember-cli-update
Sep 1, 2025
Merged

update the format of the ember-cli-update.json#10791
mansona merged 3 commits intomasterfrom
ember-cli-update

Conversation

@mansona
Copy link
Member

@mansona mansona commented Sep 1, 2025

Fundamentally this PR fixes a problem that has been surfaced by #10766

We can now have a difference in versions between ember-cli and the two app and addon blueprint packages. This means that expecting the app and addon blueprints to have a version the same as ember-cli is no longer valid.

This also sets us up to correctly update people across the vite boundary to the "old blueprints" without to much trouble 👍

@mansona mansona requested a review from a team September 1, 2025 14:37
@mansona mansona merged commit 0448044 into master Sep 1, 2025
138 of 140 checks passed
@mansona mansona deleted the ember-cli-update branch September 1, 2025 14:55
@mansona mansona mentioned this pull request Sep 1, 2025
"blueprints": [
{
"name": "<%= blueprint %>",
"outputRepo": "https://github.com/ember-cli/ember-<%= blueprint === 'app' ? 'new' : 'addon' %>-output",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It may not need to be here for ember-cli-update but we do need to make output repos for all the blueprints

{
"name": "<%= blueprint %>",
"outputRepo": "https://github.com/ember-cli/ember-<%= blueprint === 'app' ? 'new' : 'addon' %>-output",
"codemodsSource": "ember-<%= blueprint %>-codemods-manifest@1",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably do want a codemodsSource? This removes it for classic as well and so I don't believe ember-cli-update's run codemods will work without this.

They are kept here in a monorepo https://github.com/ember-cli/ember-cli-update-codemods-manifest/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants