Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace cobertura parser with patched version #360

Merged
merged 2 commits into from
Mar 25, 2022

Conversation

fschwaiger
Copy link
Contributor

This is related to, and should resolve #304.

I did not have time yet nor do I have the experience to run the tests for this.

@ryanluker ryanluker added this to the 2.10.0 milestone Mar 25, 2022
@ryanluker
Copy link
Owner

ryanluker commented Mar 25, 2022

@fschwaiger Thanks for the PR and the contribution!
I resolved the conflict with master, as there were some node versioning issues I fixed today.
Seeming the builds passed (cobertura is used for some integration level testing), I am fine to merge this in.

@ryanluker ryanluker merged commit bfa2ccb into ryanluker:master Mar 25, 2022
@ryanluker
Copy link
Owner

Thanks again for the contribution and the extension release 2.10.0 will hopefully happen within a week!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MATLAB coverage.xml with abstract classes cannot be read
2 participants