Instant App Support
dazza5000 opened this issue · 7 comments
dazza5000 commented
This plugin doesn't look like it works on Android Instant Apps
vanniktech commented
That's true. It's missing a few things here.
This project does check for the other variants.
Mind coming up with a PR to fix that?
dazza5000 commented
I would be up for it - could you provide some pointers?
Thank you!
Darran
…On Thu, Jan 25, 2018 at 4:35 AM, Niklas Baudy ***@***.***> wrote:
That's true. It's missing a few things here
<https://github.com/vanniktech/gradle-android-junit-jacoco-plugin/blob/ddf89ba2fb72098884be95b8d8a4c95c1f220e92/src/main/groovy/com/vanniktech/android/junit/jacoco/GenerationPlugin.groovy#L291>
.
This project
<https://github.com/vanniktech/gradle-code-quality-tools-plugin/blob/fa9f32d359a98675076d9f399fdf3e2d98e45272/src/main/groovy/com/vanniktech/code/quality/tools/CodeQualityToolsPlugin.groovy#L356>
does check for the other variants.
Mind coming up with a PR to fix that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#110 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABENMXAwDmgSxGiZ3aqWloL4mb3wRFpYks5tOFjcgaJpZM4RrmgJ>
.
vanniktech commented
Yeah it's the first link that I posted and then the second link shows the possible plugin ids.
dazza5000 commented
Could you give me one more hint of what I should do with those?
…On Jan 25, 2018 7:31 AM, "Niklas Baudy" ***@***.***> wrote:
Yeah it's the first link that I posted and then the second link shows the
possible plugin ids.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#110 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABENMXoQUKlZQ1JjPBu5C90jTTowXc2Pks5tOIJMgaJpZM4RrmgJ>
.
vanniktech commented
You need to add those checks in here.
dazza5000 commented
I submitted a pull request. Please let me know if there is anything I need
to include/change.
Darran
…On Thu, Jan 25, 2018 at 8:34 AM, Niklas Baudy ***@***.***> wrote:
You need to add those checks in here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#110 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABENMbEXEpDYSLQQLvbboYYROKT2MlhTks5tOJDugaJpZM4RrmgJ>
.
vanniktech commented
Fixed by #111