-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix alleycats-tests on Scala.js #3546
Conversation
23a6b87
to
c58354b
Compare
Codecov Report
@@ Coverage Diff @@
## master #3546 +/- ##
=======================================
Coverage 91.30% 91.30%
=======================================
Files 386 386
Lines 8564 8564
Branches 251 251
=======================================
Hits 7819 7819
Misses 745 745 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
* Add them to the PR build * Some minor build tweaks
c58354b
to
d47ecdc
Compare
@barambani sorry for the new commit but it was easier this way to avoid conflicts. JS tests are much faster now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess accidentally broken by #3538 and went unnoticed because it was not part of the build.