-
Notifications
You must be signed in to change notification settings - Fork 65
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
Update JaCoCo core version to support Java 10 #71
Comments
Now that Java 12 is out and being an LTS version. This issue will start to happen more often (already failing on our sonarqube analysis). |
@zageyiff I'm no member of it, but I believe this repo is just not maitained anymore. This plugin seems to have been superseeded by its fork on https://github.com/Inform-Software/sonar-groovy. If I'm not mistaken, the latest sonarqube versions already bring the forked version instead of the one on this repo. The fork fixed this bug, as you can see from the reference right above your comment. PS.: actually java 11 is the LTS. |
Thanks @acdcjunior, as this is the repo pointed out by sonarqube documentation. And you're right, the LTS version is Java 11, my bad. |
Hey there.
I'm using sonar-groovy version 1.5 with Java 10 (and gradle) and am getting the following error:
I believe this is due to the fact the only the latest version of Jacoco supports Java 10: https://github.com/jacoco/jacoco/releases/tag/v0.8.1
And the version the plugin currently uses is 0.7.5.201505241946.
Any chance we can upgrade?
I could provide the PR, if that's the only needed step.
Thanks!
The text was updated successfully, but these errors were encountered: