gradle-android-junit-jacoco-plugin icon indicating copy to clipboard operation
gradle-android-junit-jacoco-plugin copied to clipboard

Gradle 8 upgrade

Open john-tadebois opened this issue 1 year ago • 4 comments
trafficstars

I pushed this branch from themockgrateme:gradle-8-upgrade, with the fix for

POM_IMPLEMENTATION_CLASS = com.vanniktech.android.junit.jacoco.GenerationPlugin

Along with themockgraeme's changes:

https://github.com/vanniktech/gradle-android-junit-jacoco-plugin/pull/219

I forked the Laimiux Gradle 8 compatible branch of the Vanniktech Jacoco plugin and made the following changes:

Removed the Laimiux publishing calls, which are specific to their system. Upgraded the included APIs to the latest versions and made minor code changes to accommodate these. Changed the Java / JDK version to 17, for compatibility with Gradle 8. Upgraded the Gradle version to 8.

john-tadebois avatar Jan 03 '24 02:01 john-tadebois

@themockgraeme I have added the code review changes from https://github.com/vanniktech/gradle-android-junit-jacoco-plugin/pull/219.

I could not push to your branch, so I created this one. Could you please give it a quick review?

john-tadebois avatar Jan 03 '24 03:01 john-tadebois

Hey guys ....

This seem to take forever to merge, do we have any workarounds for this until this is merged?

cc @john-tadebois john

unamed000 avatar Feb 22 '24 03:02 unamed000

Hey guys ....

This seem to take forever to merge, do we have any workarounds for this until this is merged?

cc @john-tadebois john

I do not know much about the inner workings of this plugin.

My team has moved on to directly use JaCoCo.

I can still merge that PR if you do understand and can confirm in this chat that I can go ahead and merge this PR.

john-tadebois avatar Mar 21 '24 02:03 john-tadebois