Skip to content
This repository has been archived by the owner. It is now read-only.

Only report in coverage script #379

Closed
willclarktech opened this issue Dec 1, 2017 · 0 comments

Comments

1 participant
@willclarktech
Copy link
Contributor

commented Dec 1, 2017

In our npm test script we use nyc to instrument the code, but in the cover script we run the tests again, when we can just output a report based on the previous nyc output.

This issue may also need to update the jenkinsfile.

@willclarktech willclarktech self-assigned this Dec 1, 2017

@willclarktech willclarktech added this to Issues in Version 1.0.0 Dec 1, 2017

@willclarktech willclarktech moved this from Issues to Prioritised Issues in Version 1.0.0 Dec 4, 2017

willclarktech added a commit that referenced this issue Dec 5, 2017

:bowtie: Only run report in coverage script
As opposed to running the full suite of tests again

Closes #379

@willclarktech willclarktech moved this from Issues to Issues with Pull Request in Sprint Board 04-12-17 Dec 5, 2017

willclarktech added a commit that referenced this issue Dec 5, 2017

Merge pull request #385 from LiskHQ/379-coverage_report_script
Only run report in coverage script - Closes #379

@willclarktech willclarktech moved this from Prioritised Issues to Completed Issues in Version 1.0.0 Dec 5, 2017

@willclarktech willclarktech moved this from Issues with Pull Request to Completed Issues in Sprint Board 04-12-17 Dec 5, 2017

willclarktech added a commit that referenced this issue Dec 14, 2017

:bowtie: Only run report in coverage script
As opposed to running the full suite of tests again

Closes #379
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.