vanniktech / gradle-android-junit-jacoco-plugin

Gradle plugin that generates JaCoCo reports from an Android Gradle Project
http://vanniktech.com
Apache License 2.0
400 stars 72 forks source link

Plugin does not work with Gradle 8.0 #217

Open Laimiux opened 1 year ago

Laimiux commented 1 year ago

JacocoMerge task was removed (https://github.com/gradle/gradle/issues/12767) and this plugin now throwa

Failed to apply plugin 'com.vanniktech.android.junit.jacoco'.
> Could not create plugin of type 'GenerationPlugin'.
   > Could not generate a decorated class for type GenerationPlugin.
      > org/gradle/testing/jacoco/tasks/JacocoMerge
vanniktech commented 1 year ago

Honestly, this is super unmaintaned. Happy for a person to jump in

twyatt commented 1 year ago

@Laimiux any plans to get your Gradle 8 support (laimonas/gradle-8 branch) merged into this project? Or, to publish your fork publicly? 🙏

Laimiux commented 1 year ago

I've got it to work for and published it to internal maven repository (at Instacart). I'm leaving for vacation for two weeks starting tomorrow so I won't be able to work on merging this back into here. However, if anyone has bandwidth, the changes in https://github.com/Laimiux/gradle-android-junit-jacoco-plugin/tree/laimonas/gradle-8 should be easy to contribute back.

twyatt commented 1 year ago

Honestly, this is super unmaintaned.

@vanniktech if PRs are created/merged, will you continue to publish versions? Or, are you hoping someone will fork this project and publish as a new artifact?

tecruz commented 1 year ago

Any news?

themockgraeme commented 1 year ago

@vanniktech I forked Gradle 8 branch that @Laimiux created, removed some added publishing code which I suspect is specific to the Instacart repository, and updated some of the imported libraries. It's now building with Gradle 8 / Java 17.

I created a pull request, and the code can be found at:

https://github.com/themockgraeme/gradle-android-junit-jacoco-plugin/tree/gradle-8-upgrade

vramasam commented 1 year ago

Hello @themockgraeme Is there any new artifacts available for the above fix. If its there can you please provide the path

themockgraeme commented 1 year ago

Sorry, I've been away for a few months, and I'm busy with other things, so I haven't had the chance to look at it.

On Tue, 29 Aug 2023 at 21:44, vramasam @.***> wrote:

Hello @themockgraeme https://github.com/themockgraeme Is there any new artifacts available for the above fix. If its there can you please provide the path

— Reply to this email directly, view it on GitHub https://github.com/vanniktech/gradle-android-junit-jacoco-plugin/issues/217#issuecomment-1697283802, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABFSF4P37ILDX4D6CS35UDXXXI2HANCNFSM6AAAAAAU4A6R5Q . You are receiving this because you were mentioned.Message ID: <vanniktech/gradle-android-junit-jacoco-plugin/issues/217/1697283802@ github.com>

vramasam commented 12 months ago

Okay @themockgraeme . Thank you for the response. Is there any plan to update this to master and create a tag and publish?

themockgraeme commented 12 months ago

@vramasam I haven't really thought about it. I migrated the project that I'm working on to an alternative plugin.

https://github.com/gmazzo/gradle-android-test-aggregation-plugin

I'm busy for the next week and a half, but after that I might take a quick look at the project and get it compiling, but it's not something that I'm looking at taking on and maintaining.