-
Notifications
You must be signed in to change notification settings - Fork 44
Does not work with 2.13 #106
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
Comments
BTW, the version 3.1.3 does not work with 2.13 either. Complains about
|
version 4.0.1 still does not work with scala 2.13:
|
Note: scala 2.13 has method
|
Hello, I was wondering if somebody is looking into this issue. Thanks! |
@eschizoid This depends on a change to the scalac plugin, and it looks like the development on the scalac plugin has gone silent in the past few months. |
@eyalroth I think my error is not related to the problem you mentioned:
Could this be a different problem? @maiflai Maybe you know the answer since its failing here?
Thanks! |
Is there any update on this issue? I am running into the same problem with scoverage plugin version 4.0.1 and scala version 2.13.1. |
And...what about writing the Writer in Scala? Wouldn't that fix the issue? |
is there a release planed to fix this issue? |
I think using Scala in the Gradle plugin makes the plugin classpath difficult; this has been discussed before. A PR for this has been merged in scalac-scoverage-plugin but we now need a release there before the Gradle plugin can use it. Stu. |
It will require to compile and publish the plugin for every major Scala version, which makes both the development and usage of it a bit more complicated. |
@maiflai scalac-scoverage-plugin 1.4.2 has been released. |
so has the updated version of this plugin - please could you try 5.0.0? |
I updated to Scoverage 5.0.0 and the scalac-scoverage-plugin 1.4.2. I'm not getting any errors (yay!) but neither do I get any output. Though I would be hard pressed to prove it wasn't a problem with the configuration. Any Scoverage task that I try and run gets skipped. build.gradle:
Here's the output from the relevant section with --debug: 2020-12-14T23:46:49.945-0500 [QUIET] [system.out] Run completed in 9 seconds, 748 milliseconds. |
Sorry for the delay.
Please could you try changing test {
suite 'org.primitive.MasterSuite'
} to tasks.withType(Test) {
suite 'org.primitive.MasterSuite'
} Or perhaps you could just try adding testScoverage {
suite 'org.primitive.MasterSuite'
} |
Now worries, I've got plenty of things to keep me busy. I appreciate the assistance. So that last one, adding:
A problem occurred evaluating root project 'primitive'.
The other suggestion gets past the build file verification, but :compileScoverageScala fails with 100 compile errors - it isn't recognizing references to some classes in my code. My first guess would be that it sounds like a class path problem, but a cursory glance the classpath seems ok (it's long enough that it's hard to know) 2020-12-27T23:03:47.898-0500 [DEBUG] [org.gradle.api.internal.tasks.scala.ZincScalaCompilerFactory] [zinc] The Scala compiler is invoked with: If you want to experiment with it, the most recent version that compiles and all the tests pass is on GitHub: https://github.com/dmclean62/primitive |
I am hitting this issue as well on Gradle 7.14, with v7.0.0 of the Scoverage gradle plugin, and v2.13.7 of scala and Java 11.0.13. This still seems to be an unresolved issue. |
I think this particular issue is now being used for several different problems. If you are using the If that is not possible because of a feature gap, then perhaps you could raise a new issue here with a simple reproduction of your particular problem? Thanks, |
I've been digging a little deeper into this issue. It looks to me that the ScoverageAggregate.groovy could benefit from using JavaConversions as well, to prevent having to depend on an upstream overloaded method, which until recently had no test coverage. I also think calling the upstream |
Hello,
The plugin does not work with scala 2.13 because scala.collection.JavaConversions is not there any more.
build.gradle (removed unrelated stuff)
running with
./gradlew reportTestScoverage --stacktrace
and getting
The text was updated successfully, but these errors were encountered: