-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: replace airtap with wtr #1722
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1722 +/- ##
=======================================
Coverage 81.03% 81.03%
=======================================
Files 21 21
Lines 1366 1366
Branches 323 323
=======================================
Hits 1107 1107
Misses 182 182
Partials 77 77 ☔ View full report in Codecov by Sentry. |
@BertKleewein @vishnureddy17 @mcollina This has been painful but I think worth it. Now browser tests are much better, they will run on chromium, firefox and webkit using both ws and wss 🎉 |
Amazing work, thank you! |
Another long awaited change is: #1731 This will also make tests with wtr more streightforward as it only support esm |
Fixes #1655