-
Notifications
You must be signed in to change notification settings - Fork 714
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 test coverage reports #523
Comments
The problem is with gulp-mocha ^4.0. sindresorhus/gulp-mocha#159 |
Just as a general idea and to prevent things like this, maybe we should lock the dependency versions? I mean we have greenkeeper enabled. |
@lholznagel I know but this is an odd one because 3.0 and 4.0 is a breaking change. But the build is successful and the coverage reports are not available :( |
@remojansen What if you check that the coverage report is not empty?
|
@AltekkeE That sounds like a good way to prevent this issue in the future. Thanks for sharing it 👍 |
Rolled back to 3.0.1 will wait until gulp-mocha fixes the problems. |
The cov reports are not working. More info at inversify/inversify-express-utils#41 (comment)
The text was updated successfully, but these errors were encountered: