Skip to content
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

Update to the Ember 4.12 LTS blueprint #34

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Feb 19, 2024

This updates to the 4.12 blueprints and does the minimum to make the linters not complain. The silenced linting errors can then be fixed in a future pr.

It's better to simply run `npx ember-cli-update` so the latest version is always installed.
This resolves all autofixable linting errors.
This tool adds linting ignore comments to all code that triggers errors, so we can fix those in the future without having to globally ignore certain rules.
lint-to-the-future doesn't seem to support adding ignore comments to the test templates, so we resolved those manually.
We don't have any .css files so stylelint throws an error without that flag.
"ember-auto-import": "^1.12.0",
"ember-cli-babel": "^7.26.10",
"ember-cli-htmlbars": "^5.7.2",
"ember-auto-import": "^2.6.3",
Copy link
Contributor Author

@Windvis Windvis Feb 20, 2024

Choose a reason for hiding this comment

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

This could be considered a breaking change, since the consuming app also needs to have ember-auto-import v2 installed. This is a requirement for v4, so only 3.x apps might still have ember-auto-import v1. I'm not sure what the support policy is here.

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.

1 participant