Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upload coverage from GHA #480

Merged
merged 2 commits into from
Nov 14, 2020
Merged

Upload coverage from GHA #480

merged 2 commits into from
Nov 14, 2020

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Nov 14, 2020

Subset of #479.


We already have tests on GitHub Actions, but are missing sending coverage to Codecov. Earlier it was tricky to do, but Codecov now have an action for this; added here.


This also fixes a pytest warning:

=============================== warnings summary ===============================
.tox/py37/lib/python3.7/site-packages/_pytest/config/__init__.py:1230
  /home/travis/build/hugovk/tablib/.tox/py37/lib/python3.7/site-packages/_pytest/config/__init__.py:1230: PytestConfigWarning: Unknown config option: python_paths
  
    self._warn_or_fail_if_strict("Unknown config option: {}\n".format(key))
-- Docs: https://docs.pytest.org/en/stable/warnings.html

@codecov
Copy link

codecov bot commented Nov 14, 2020

Codecov Report

Merging #480 (f91f369) into master (615e308) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #480   +/-   ##
=======================================
  Coverage   90.66%   90.66%           
=======================================
  Files          28       28           
  Lines        2613     2613           
=======================================
  Hits         2369     2369           
  Misses        244      244           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 615e308...f91f369. Read the comment docs.

@hugovk hugovk merged commit 22c88de into jazzband:master Nov 14, 2020
@hugovk hugovk deleted the gha-coverage branch November 14, 2020 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants