search

coverallsapp / coverage-reporter

Self-contained, universal coverage uploader binary. Under development.
https://coveralls.io
MIT License
45 stars 14 forks source link

Does this orb support the coveralls or coveralls+ formats? #71

Closed josegonzalez closed 1 year ago

josegonzalez commented 1 year ago

Question

Does this orb support the coveralls or coveralls+ formats? From reading the readme/source, it seems like no, but I just want to be sure.

mrexox commented 1 year ago

No, currently it doesn't support coveralls/coveralls+ formats. But I am interested in adding the support for them. Can you share the tools you use to generate reports in these formats? It would be helpful.

josegonzalez commented 1 year ago

I am using grcov for a rust-based program to generate those reports. I'm migrating from the old version of the coveralls orb that supports a node project for those formats, and wanted to see if maybe switching from lcov would fix my parsing issues.

mrexox commented 1 year ago

I have added the support for coveralls format of grcov: https://github.com/coverallsapp/coverage-reporter/pull/79. It will be available with the v0.4.3 release.

josegonzalez commented 1 year ago

Should it also support coveralls+ (I don't know what the difference is tbh)?

mrexox commented 1 year ago

I've read the code for coveralls+ format - it adds functions field, but I'm afraid it doesn't get processed by Coveralls now. Maybe it was a few years ago (the code for coveralls+ format is written 6 years ago btw). So, basically coveralls+ adds nothing really special.

josegonzalez commented 1 year ago

Thanks for the response!