search

ssec-jhu / dplutils

Distributed(Data, Discovery) Pipeline Uitilities
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

Restoring codecov action #9

Closed amitschang closed 10 months ago

amitschang commented 10 months ago

@jamienoss, thanks! I thought we had that already by default (and seems we do in base-template). Perhaps though we should just always do both html and xml

jamienoss commented 10 months ago

base-template useses xml https://github.com/ssec-jhu/base-template/blob/main/tox.ini#L30 I'm down for using both as html is better for viewing local reports. In the very beginning, it was both, but I seem to remember something not working though...

codecov[bot] commented 10 months ago

Welcome to Codecov :tada:

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

amitschang commented 10 months ago

@jamienoss , not sure. This change that contains both looks like it works both locally and with codecov. Also, note that the doc thing is expected to fail, but fix is already in the pipe in #8