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

pinning @types/qunit to fix ci #2159

Merged
merged 1 commit into from
Oct 31, 2024
Merged

pinning @types/qunit to fix ci #2159

merged 1 commit into from
Oct 31, 2024

Conversation

ef4
Copy link
Contributor

@ef4 ef4 commented Oct 31, 2024

@types/qunit 2.19.11 is incompatible with ember-qunit right now.

@SergeAstapov
Copy link
Collaborator

SergeAstapov commented Oct 31, 2024

@ef4 there was release of ember-qunit with fix https://github.com/emberjs/ember-qunit/releases/tag/v8.1.1 just couple hours ago

@ef4
Copy link
Contributor Author

ef4 commented Oct 31, 2024

Oh, that's good.

But also I see our release scenarios are explicitly using ember-qunit 7 instead of 8. We'll need to sort out why and whether we're ready to only test 8.

@SergeAstapov
Copy link
Collaborator

good point! we may want to release patch for ember-qunit v7 then

@ef4 ef4 merged commit f879e60 into stable Oct 31, 2024
221 checks passed
@ef4 ef4 deleted the fix-qunit-types branch October 31, 2024 19:33
@github-actions github-actions bot mentioned this pull request Oct 31, 2024
@ef4 ef4 added the internal label Oct 31, 2024
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.

2 participants