DM-42534: Upload coverage report to Codecov #3
Conversation
.coveragerc
Outdated
| @@ -0,0 +1,8 @@ | |||
| [report] | |||
There was a problem hiding this comment.
Can you put this in pyproject.toml since that's where we are going to end up and it saves proliferating lots of bespoke config files.
There was a problem hiding this comment.
Done. And I moved the config from .flake8 to setup.cfg.
There was a problem hiding this comment.
I decided against moving to setup.cfg and chose to keep .flake8 to keep in sync with the repo upstream.
.github/workflows/lsst-tests.yml
Outdated
| - name: Upload coverage reports to Codecov | ||
| uses: codecov/codecov-action@v3 | ||
| env: | ||
| CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} |
There was a problem hiding this comment.
I think you can remove this when it moves to lsst org (and I don't see why we can't enable lsst-dm org like lsst org).
There was a problem hiding this comment.
I switched it to report in XML format and upload the file. No token required.
arunkannawadi
force-pushed
the
tickets/DM-42534
branch
2 times, most recently
from
7096311
to
c041262
Compare
Always glad when someone learns something new from a PR! |
arunkannawadi
force-pushed
the
tickets/DM-42534
branch
from
8e8a1e8
to
ac6b88b
Compare
arunkannawadi
force-pushed
the
tickets/DM-42534
branch
2 times, most recently
from
efb7a7a
to
f3ce873
Compare
DM-42534: Upload coverage report to Codecov
DM-42534: Upload coverage report to Codecov
No description provided.