Skip to content

fix(build): ember-auto-import@2 compatibility#30

Open
buschtoens wants to merge 1 commit intoworkmanw:masterfrom
buschtoens:patch-1
Open

fix(build): ember-auto-import@2 compatibility#30
buschtoens wants to merge 1 commit intoworkmanw:masterfrom
buschtoens:patch-1

Conversation

@buschtoens
Copy link
Copy Markdown

Makes this addon compatible with ember-auto-import@2 host apps.

Closes #25.

This is not the ideal solution, but the most trivial one.

As this addon is considered dormant and its features are set to be upstreamed into ember-qunit, I don't expect the owner @workmanw to cut a new release. Instead affected users may use patch-package to apply this fix locally.

@ef4
Copy link
Copy Markdown

ef4 commented Jan 31, 2022

The other thing I'd recommend is dropping this auto-registration entirely and telling users to do this import themselves, from their test-setup.js file.

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