Merge lp:~jelmer/brz/coverage-coverage into lp:brz
Proposed by
Jelmer Vernooij
Status: | Merged |
---|---|
Approved by: | Jelmer Vernooij |
Approved revision: | no longer in the source branch. |
Merge reported by: | The Breezy Bot |
Merged at revision: | not available |
Proposed branch: | lp:~jelmer/brz/coverage-coverage |
Merge into: | lp:brz |
Diff against target: |
157 lines (+26/-23) 6 files modified
.bzrignore (+1/-0) .coveragerc (+1/-0) .travis.yml (+2/-3) breezy/commands.py (+15/-19) breezy/plugin.py (+1/-1) breezy/tests/__init__.py (+6/-0) |
To merge this branch: | bzr merge lp:~jelmer/brz/coverage-coverage |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Packman | Approve | ||
Jelmer Vernooij | Approve | ||
Review via email: mp+345192@code.launchpad.net |
Commit message
Switch --coverage to use coverage module.
Description of the change
This should make it easier to integrate with e.g. codecov.io; it also allows more easily generating HTML coverage output (rather than the text output that tracer generates).
To post a comment you must log in.
Looks good to me. The envvar setting is a little ugly but interface is otherwise close enough and this is the module I use on other projects.