-
Notifications
You must be signed in to change notification settings - Fork 404
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
fixes #1419 implement failed tests count in summary output #1420
fixes #1419 implement failed tests count in summary output #1420
Conversation
locally the all tests passed with the integration tests, probably it's something not related to the current code changes
|
@Mifrill could you rebase your branch? The tests are fixed in the latest master. |
8844add
to
95fc9b2
Compare
is it better to probably use the same approach as for TapReporter?:
|
Yes, if you could do this change, it would probably be better to keep them consistent. |
4b92649
to
705290a
Compare
705290a
to
e5bb8cc
Compare
@johanneswuerbach thank you! |
Released as https://github.com/testem/testem/releases/tag/v3.3.0, thank you 👏 |
closes #1419