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

Make the Cobertura profile inactive by default #59

Closed
lennartj opened this issue Dec 2, 2017 · 7 comments
Closed

Make the Cobertura profile inactive by default #59

lennartj opened this issue Dec 2, 2017 · 7 comments

Comments

@lennartj
Copy link
Member

lennartj commented Dec 2, 2017

... and consider moving to Clover instead. It works well with Java 8.

@khmarbaise
Copy link
Member

khmarbaise commented Jan 9, 2018

Also an option would be JaCoCo ?
And yes we need to leave cobertura behind..

@slachiewicz
Copy link
Member

If this plugin is enabled, you can not run site generation under Java 12 (tested with extra-enforcer-rules)

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.7.1:site (default-site) on project extra-enforcer-rules: failed to get report for org.codehaus.mojo:cobertura-maven-plugin: Plugin org.codehaus.mojo:cobertura-maven-plugin:2.7 or one of its dependencies could not be resolved: Could not find artifact com.sun:tools:jar:0 at specified path jdk-11.0.1/../lib/tools.jar -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.7.1:site (default-site) on project extra-enforcer-rules: failed to get report for org.codehaus.mojo:cobertura-maven-plugin

@andham
Copy link
Member

andham commented Jan 18, 2020

What should we do here? Is JaCoCo they way to go nowadays?

@andham
Copy link
Member

andham commented Jan 18, 2020

I've added openjdk11 to the travis ci build and it now shows this problem with cobertura by failing all opendjk 11 builds.

@andham
Copy link
Member

andham commented Jan 21, 2020

Fixed by #79, which removes cobertura-m-p all together.

@andham andham closed this as completed Jan 21, 2020
@mfriedenhagen
Copy link
Member

mfriedenhagen commented Jul 5, 2020

Fixed by #7, which removes cobertura-m-p all together.

@andham, should this be #77 :-)

@andham
Copy link
Member

andham commented Jul 5, 2020

@mfriedenhagen I think it should be #79, which is the fix (PR). I blame my keyboard for the typo.
I've updated the comment so that mention links are correct.

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

No branches or pull requests

5 participants