search

parroty / excoveralls

Coverage report tool for Elixir with coveralls.io integration.
MIT License
820 stars 178 forks source link

cobertura task #302

Closed albertored closed 1 year ago

albertored commented 1 year ago

A taks is added for generating XML Cobertura reports

This is useful for instance to enable the Test coverage visualization feature of GitLab.

coveralls commented 1 year ago

Coverage Status

Coverage: 88.667% (+0.8%) from 87.828% when pulling fe0d7ab54b6ab478cc39698b8fac236c34074eae on albertored:cobertura into 6890fdc60bdeef3f718bac7c53f3a4ba4b3523fe on parroty:master.

coveralls commented 1 year ago

Coverage Status

Coverage: 88.648% (+0.8%) from 87.828% when pulling 550311623adc366b8de6f8eb8e55064ec79f973d on albertored:cobertura into 6890fdc60bdeef3f718bac7c53f3a4ba4b3523fe on parroty:master.

parroty commented 1 year ago

This is useful for instance to enable the Test coverage visualization feature of GitLab.

Thanks for the PR 🙇 . I wasn't aware of this type of usage.

I am having certain concerns around maintenance for adding new types, but it seems the format is stable (not updated recently). If it doesn't break other features, and if not-constant updates are required, it may be one option to include this subcommand.

Do you have any comments / insights around the compatibility / stability for this feature support? (considering the maintenance).

albertored commented 1 year ago

The cobertura tool seems very stable (if not freezed). The last commit (as you were noticing) is from 2015 and the last edit to the DTD schema was made in 2013. I don't expect any change to the excoveralls code that exports this report in the near future.

parroty commented 1 year ago

Thanks for the additional comments:bow: