Skip to content

Conversation

@ef4
Copy link
Contributor

@ef4 ef4 commented Nov 10, 2025

babel-plugin-ember-template-compilation was downgraded from 3.x to 2.x only to keep eslint happy. That's not good, we should be shipping the current major.

Also, everybody still has 3.x also, because @embroider/core depend on it. The mixing is inconsistent.

babel-plugin-ember-template-compilation was downgraded from 3.x to 2.x only to keep eslint happy.  That's not good, we should be shipping the current major.

Also, everybody still has 3.x also, because `@embroider/core` depend on it. The mixing is inconsistent.
@ef4 ef4 added the bug Something isn't working label Nov 10, 2025
@ef4
Copy link
Contributor Author

ef4 commented Nov 11, 2025

This will also need: ember-tooling/ember-eslint-parser#137

@NullVoxPopuli
Copy link
Contributor

This will also need: ember-tooling/ember-eslint-parser#137

I don't think this was enough 🤔 it helped! though as I see far fewer errors in a local project

@evoactivity
Copy link
Contributor

evoactivity commented Nov 12, 2025

@NullVoxPopuli @ef4 should just need to remove project: true from the eslint configs to get this passing, the reason it was added has been fixed and released in ember-eslint-parser.

@NullVoxPopuli NullVoxPopuli merged commit 6ff3365 into main Nov 12, 2025
5 of 6 checks passed
@NullVoxPopuli NullVoxPopuli deleted the dont-downgrade-etc branch November 12, 2025 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants