Skip to content

Report coverage to Codecov#386

Merged
jakirkham merged 6 commits intozarr-developers:mainfrom
jakirkham:config_codecov
Nov 4, 2022
Merged

Report coverage to Codecov#386
jakirkham merged 6 commits intozarr-developers:mainfrom
jakirkham:config_codecov

Conversation

@jakirkham
Copy link
Copy Markdown
Member

Setup coverage reporting with Codecov.

TODO:

  • Unit tests and/or doctests in docstrings
  • Tests pass locally
  • Docstrings and API docs for any new/modified user-facing classes and functions
  • Changes documented in docs/release.rst
  • Docs build locally
  • GitHub Actions CI passes
  • Test coverage to 100% (Coveralls passes)

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 4, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@6facbba). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #386   +/-   ##
=======================================
  Coverage        ?   99.57%           
=======================================
  Files           ?       54           
  Lines           ?     2099           
  Branches        ?        0           
=======================================
  Hits            ?     2090           
  Misses          ?        9           
  Partials        ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jakirkham jakirkham requested a review from joshmoore November 4, 2022 09:05
@joshmoore
Copy link
Copy Markdown
Member

Hmmm....something's gone wrong with the tests here.

@jakirkham
Copy link
Copy Markdown
Member Author

Yeah they ran, but the status wasn't reported in the PR. Not sure what is up there. Let's try again 🍀

@jakirkham jakirkham enabled auto-merge (squash) November 4, 2022 20:16
@jakirkham
Copy link
Copy Markdown
Member Author

Seems to be working. Going to get this in so we can start collecting coverage data. Happy to follow up on anything after.

@jakirkham jakirkham merged commit ff65b71 into zarr-developers:main Nov 4, 2022
@jakirkham jakirkham deleted the config_codecov branch November 4, 2022 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