[tahoe-lafs-trac-stream] [Tahoe-LAFS] #3384: Unpin coverage
Tahoe-LAFS
trac at tahoe-lafs.org
Wed Aug 19 15:22:18 UTC 2020
#3384: Unpin coverage
------------------------------------+-----------------------
Reporter: sajith | Owner:
Type: defect | Status: new
Priority: normal | Milestone: undecided
Component: dev-infrastructure | Version: n/a
Resolution: | Keywords:
Launchpad Bug: |
------------------------------------+-----------------------
Description changed by sajith:
Old description:
> #3267 pinned some dependencies including coverage, because integration
> tests were failing on CI. It isn't clear if coverage still needs to
> remain pinned at ~= 4.5.
>
> Let us unpin coverage and see what happens in CI!
New description:
#3267 pinned some dependencies including coverage, because integration
tests were failing on CI. It isn't clear if coverage still needs to
remain pinned at ~= 4.5.
Let us unpin coverage and see what happens in CI!
For more context, the follow-up ticket to this would be #3385, which will
attempt to use [[https://coveralls.io|coveralls.io]] for code coverage
checks. We are not entirely satisfied with
[[https://codecov.io|codecov.io]], which we currently use. For example,
it is not always clear why codecov checks turn disapprove of certain PRs,
even when test coverage remains unchanged.
Now, in order to upload coverage reports to codecov.io, we will need to
use [[https://github.com/coveralls-clients/coveralls-python|coveralls-
python]], which requires that coverage reports should be in coverage 5.0
format. Which is the real motivation for unpinning coverage.
(See coverage
[[https://coverage.readthedocs.io/en/coverage-5.2.1/changes.html#version-5-0a2-2018-09-03|changelog]]:
"Coverage’s data storage has changed. In version 4.x, .coverage files were
basically JSON. Now, they are SQLite databases.")
--
--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3384#comment:3>
Tahoe-LAFS <https://Tahoe-LAFS.org>
secure decentralized storage
More information about the tahoe-lafs-trac-stream
mailing list