Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

excluding test code itself from coverage #395

Merged
merged 1 commit into from Feb 10, 2016
Merged

Conversation

NikVolf
Copy link
Contributor

@NikVolf NikVolf commented Feb 10, 2016

No description provided.

@NikVolf NikVolf added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Feb 10, 2016
gavofyork pushed a commit that referenced this pull request Feb 10, 2016
excluding test code itself from coverage
@gavofyork gavofyork merged commit f0e26e3 into master Feb 10, 2016
./kcov-master/tmp/usr/local/bin/kcov --exclude-pattern /.cargo,/root/.multirust,src/tests,util/json-tests,util/src/network/tests,sync/src/tests,ethcore/src/tests,ethcore/src/evm/tests target/kcov target/debug/deps/ethash-* &&
./kcov-master/tmp/usr/local/bin/kcov --exclude-pattern /.cargo,/root/.multirust,src/tests,util/json-tests,util/src/network/tests,sync/src/tests,ethcore/src/tests,ethcore/src/evm/tests target/kcov target/debug/deps/ethcore-* &&
./kcov-master/tmp/usr/local/bin/kcov --exclude-pattern /.cargo,/root/.multirust,src/tests,util/json-tests,util/src/network/tests,sync/src/tests,ethcore/src/tests,ethcore/src/evm/tests target/kcov target/debug/deps/ethsync-* &&
./kcov-master/tmp/usr/local/bin/kcov --exclude-pattern /.cargo,/root/.multirust,src/tests,util/json-tests,util/src/network/tests,sync/src/tests,ethcore/src/tests,ethcore/src/evm/tests target/kcov target/debug/deps/ethcore_rpc-* &&
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this somehow broke file mappings: https://coveralls.io/builds/5027683/source?filename=util%2Fsrc%2Fnetwork%2Ferror.rs

We can no longer see source code preview

@debris debris deleted the coverage-reasonable branch February 17, 2016 16:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants