-
Notifications
You must be signed in to change notification settings - Fork 18
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
Fix Jacoco coverage repots in Sonar #136
Conversation
Now that inject and plexus are consolidated, they should not step on each other toe, and write to top level target directory the report.
So, coverage is not picked up by Sonar... unsure why 🤔 |
Am giving up this fight with Sonar, anyone any clue? |
Is this really that hard to achieve with Maven as outlined in https://github.com/jacoco/jacoco/wiki/MavenMultiModule? I think implementing a https://maven.apache.org/ref/3.9.7/maven-core/apidocs/org/apache/maven/execution/AbstractExecutionListener.html in Jacoco should allow to get rid of this additional module. WDYT @cstamas? |
No idea. Imho this is not jacoco (if you build locally all the files and reports are properly in place: per module reports but also the aggrgated one). It is more about sonat not picking up them, like it have clue where to look? |
|
Now that inject and plexus are consolidated, they should not step on each other toe, and write to top level target directory the report.
This does NOT fix Sonar, that for me, unknown reason, does not pick up generated Jacoco reports. Still, the reports on local builds are there and are usable.