search

typst / ecow

Compact, clone-on-write vector and string.
Apache License 2.0
205 stars 16 forks source link

Publish coverage to codecov? #15

Closed CosmicHorrorDev closed 1 year ago

CosmicHorrorDev commented 1 year ago

Wanted to check before I go ahead and open a PR, but would you be fine with a CI job to publish code coverage information to codecov? I'm planning on adding property testing and fuzzing, and having code coverage information published would be helpful (and you can get a cool little badge for it)

laurmaedje commented 1 year ago

Fuzzing sounds great. I wanted to do it, but am too busy unfortunately. I will take a look at codecov.

laurmaedje commented 1 year ago

What would uploading the coverage to codecov give us in contrast to just running cargo-tarpaulin locally?

CosmicHorrorDev commented 1 year ago

A better interface for looking through coverage, a way of tracking changes over time (catching if a change significantly changes coverage when it shouldn't), and a flashy badge for anyone who is interested in the coverage of their deps

None of those are really crucial, but surely still nice-to-haves

laurmaedje commented 1 year ago

If it helps with yor testing plan, I'm on board with it!

CosmicHorrorDev commented 1 year ago

Awesome! I'll send a PR later today :D