search

csvalpha / sofia

S.O.F.I.A. - Streepsysteem der C.S.V. Alpha
https://streep.csvalpha.nl
MIT License
5 stars 2 forks source link

feat(tests): add code coverage reporting #891

Closed guidojw closed 4 days ago

guidojw commented 1 year ago

Summary

Adds code coverage reporting with Codecov.

requires https://github.com/csvalpha/sofia/pull/945

codecov[bot] commented 1 year ago

Welcome to Codecov :tada:

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

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

lodewiges commented 3 weeks ago

this first needs a fix similair to amber-api https://github.com/csvalpha/amber-api/pull/441 https://github.com/csvalpha/amber-api/pull/444

guidojw commented 1 week ago

@lodewiges fixed

lodewiges commented 1 week ago

This doesnt fixes it if i am correct or maybe it does? I already made a seprate PR for it: https://github.com/csvalpha/sofia/pull/945 so either way it should be alright

guidojw commented 1 week ago

It does fix it, I missed some changes copying it from the other similar prs before but it should be good now