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

[Test]: Show support for qunit 5 #275

Closed
wants to merge 7 commits into from
Closed

[Test]: Show support for qunit 5 #275

wants to merge 7 commits into from

Conversation

snewcomer
Copy link
Collaborator

@snewcomer snewcomer commented Jan 31, 2021

close #273

@cah-brian-gantzler
Copy link

ember-qunit 5 is a breaking change. You have to make sure you have added some dom to the test index.html (it is harmless if you leave it in for ember-qunit 4) and some dependancies are not longer provided but are peer, so you need to specify them yourself (again doesnt hurt if you specify them in the ember-qunit4 code).

@snewcomer snewcomer self-assigned this Feb 19, 2021
@snewcomer snewcomer changed the title [Test]: qunit 5 [Test]: Show support for qunit 5 Feb 19, 2021
@snewcomer snewcomer closed this Oct 26, 2021
@snewcomer snewcomer deleted the sn/qunit-5 branch October 26, 2021 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to version 4.x leads to test errors
2 participants