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.

Sign up for GitHub

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

Jump to bottom

Update jacoco #59

Closed
wants to merge 1 commit into from
Closed

Update jacoco #59

wants to merge 1 commit into from

Conversation

mezzatto
Copy link

No description provided.

@henryju
Copy link
Member

henryju commented Apr 24, 2019

Hi,

Any reason for this change?

@mezzatto
Copy link
Author

We have a java 11 project that was unable to use the sonar scanner plugin after being upgraded from java 8. In the latest jacoco release (0.8.3) the official support for java 11 was added (jacoco/jacoco#760).

@henryju
Copy link
Member

henryju commented Apr 25, 2019

Then you have to update JaCoCo in your project build.gradle. The version you changed here is the one used to measure coverage of the scanner source code itself. And since we are still building with Java 8, we don't really need that now.

@henryju henryju closed this Apr 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@mezzatto @henryju