-
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
Stop forcing Jetpack Compose compiler plugin #704
Conversation
Thanks for the contribution! Before we can merge this, we need @koalaparadise to sign the Salesforce Inc. Contributor License Agreement. |
At the moment, the Jetpack Compose compiler plugin is force used on all Kotlin tasks. The JC compiler only works for Android and Desktop, which means that downstream users can't use the artifacts on other KMP targets (such as iOS). The JB Compose Multiplatform Gradle plugin auto sets up their Compose compiler plugin, so there's no need for the override. The Jetpack Compose compiler is already setup for Android projects through the `commonAndroidConfig`, so again no need for the override.
f56652b
to
99162b2
Compare
Thanks for the contribution! Before we can merge this, we need @chrisbanes to sign the Salesforce Inc. Contributor License Agreement. |
We need slackhq/circuit#704 for iOS and Native support.
Just in case anyone needs this immediately, I've published a fork containing this PR: |
We need slackhq/circuit#704 for iOS and Native support.
Going to backport some changes from #643 to better support this |
Continuation from #704 --------- Co-authored-by: Chris Banes <chris@banes.me>
We need slackhq/circuit#704 for iOS and Native support.
We need slackhq/circuit#704 for iOS and Native support.
At the moment, the Jetpack Compose compiler plugin is force used on all Kotlin tasks. The JC compiler only works for Android and Desktop, which means that downstream users can't use the artifacts on other KMP targets (such as iOS).
The JB Compose Multiplatform Gradle plugin auto sets up their Compose compiler plugin, so there's no need for the override. The Jetpack Compose compiler is already setup for Android projects through the
commonAndroidConfig
, so again no need for the override.