search

ralexstokes / ssz-rs

Implementation of ethereum's `ssz`
Apache License 2.0
103 stars 41 forks source link

Adds code coverage workflow and badge. #59

Closed claravanstaden closed 1 year ago

claravanstaden commented 1 year ago

@ralexstokes to finish this PR the ssz-rs repo would need the CodeCov app to be added.

ralexstokes commented 1 year ago

I've added the code cov app for this repo. And generally thumbs up on this approach

it looks like this PR uses tarpaulin, which I haven't used before but don't see any issues yet -- ill go do some more research but if you want to send over any resources that would be appreciated!

codecov[bot] commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@eb73ad8). Click here to learn what that means. Patch has no changes to coverable lines.

:exclamation: Current head 9cc2fb1 differs from pull request most recent head c72e942. Consider uploading reports for the commit c72e942 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #59 +/- ## ======================================= Coverage ? 59.91% ======================================= Files ? 20 Lines ? 953 Branches ? 0 ======================================= Hits ? 571 Misses ? 382 Partials ? 0 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

claravanstaden commented 1 year ago

@ralexstokes thanks for reviewing! I applied the changes. 😄