Skip to content

Test earlier embers#93

Open
NullVoxPopuli wants to merge 4 commits intomasterfrom
test-earlier-embers
Open

Test earlier embers#93
NullVoxPopuli wants to merge 4 commits intomasterfrom
test-earlier-embers

Conversation

@NullVoxPopuli
Copy link
Copy Markdown
Contributor

No description provided.

@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review April 15, 2026 03:49
@NullVoxPopuli
Copy link
Copy Markdown
Contributor Author

NullVoxPopuli commented Apr 15, 2026

the "fix" (some macros code for accessing @glimmer/validator) isn't neeeded in this project. so that means
make sure that:

  • ember-auto-import is up to date
  • embroider is up to date (for your series, v3 or v4)

This reverts commit ad75f5a.
Comment thread .try.mjs
scenarios: [
{
name: 'ember-lts-5.8',
name: 'ember-lts-3.28',
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I don't think 3.28 support is needed since it was dropped in v3?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don' think so? it just says ember-cli was updated?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The description of PR #81 explicitly mentions that it drops support for ember < 4.12, which is why it was marked as breaking?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ah ok, I was just reading the CHANGELOG.

But, it seems they dropped support only because the blueprint made it harder to support. It's very easy now

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fair enough, but if you widen the range again I would also update the versions in the readme. It also mentions 4.12 at the moment 😄 .

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants