Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Remove redundant test run from coverall support #298

Closed
ccowell opened this issue Feb 10, 2019 · 1 comment
Closed

Remove redundant test run from coverall support #298

ccowell opened this issue Feb 10, 2019 · 1 comment
Assignees
@ccowell

Comments

@ccowell
Copy link
Contributor

ccowell commented Feb 10, 2019 •
edited
Loading

Travis CI runs for over 10 minutes for successful ganache-core updates. Removing the extra mocha run would reduce the run time by 35-50%.

Expected Behavior

Tests should be ran once and the saved test data should be sent to coveralls.io.

Current Behavior

Travis CI runs mocha twice for test validation and again for the coverage report.
@ccowell ccowell self-assigned this Feb 10, 2019
@ccowell
Copy link
Contributor Author

ccowell commented Feb 20, 2019 •
edited
Loading

This was merged at eb95f9d

@ccowell ccowell closed this as completed Feb 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ccowell ccowell

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ccowell