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 ember-auto-import to 2.x #921

Merged
merged 2 commits into from Jul 25, 2022

Conversation

mydea
Copy link
Contributor

@mydea mydea commented Feb 2, 2022

This updates ember-auto-import to 2.x (ember-qunit is the last addon in our app still being tied to v1).

@mydea mydea force-pushed the fn/ember-auto-import-v2-node-12 branch 2 times, most recently from 2097df8 to 4ef3a48 Compare February 2, 2022 15:39
@mydea mydea force-pushed the fn/ember-auto-import-v2-node-12 branch from 4ef3a48 to d38bb7e Compare April 11, 2022 07:06
@rwjblue
Copy link
Member

rwjblue commented Apr 12, 2022

Let's do a dedicated PR to drop support for Node < 14 (the engines field should be 14.* || 16.* || >= 17), then update this PR to only bump ember-auto-import to 2.x.

@locks
Copy link
Contributor

locks commented Jul 25, 2022

@mydea hi! do you have some time to resolve the conflicts?

@mydea mydea changed the title Update ember-auto-import to 2.x & drop support for Node 10 Update ember-auto-import to 2.x Jul 25, 2022
@mydea mydea force-pushed the fn/ember-auto-import-v2-node-12 branch from d38bb7e to 3806e10 Compare July 25, 2022 13:00
@mydea
Copy link
Contributor Author

mydea commented Jul 25, 2022

I have rebased this on master (and removed all references to dropping any node version from this PR)!

@locks
Copy link
Contributor

locks commented Jul 25, 2022

Thank you for the quick turnaround!

@locks locks merged commit 364ed69 into emberjs:master Jul 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants