Skip to content
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

gradle-plugin: Let Gradle provide its Kotlin DSL for compose #2215

Merged
merged 1 commit into from
Sep 22, 2022
Merged

gradle-plugin: Let Gradle provide its Kotlin DSL for compose #2215

merged 1 commit into from
Sep 22, 2022

Conversation

Omico
Copy link
Contributor

@Omico Omico commented Jul 28, 2022

Now we could let Gradle provide its Kotlin DSL for compose.

Before and after:

  import org.jetbrains.compose.ExperimentalComposeLibrary
- import org.jetbrains.compose.compose // This is no longer required.
  plugins {
      id("org.jetbrains.compose")
      kotlin("multiplatform")
  }

  kotlin {
      sourceSets {
          commonMain {
              dependencies {
                  api(compose.foundation)
                  api(compose.materialIconsExtended)
                  api(compose.preview)
                  api(compose.runtime)
                  api(compose.ui)
              }
          }
      }
  }

@igordmn
Copy link
Collaborator

igordmn commented Sep 22, 2022

Thanks! It looks much better.

@igordmn igordmn merged commit 99b7a98 into JetBrains:master Sep 22, 2022
@igordmn igordmn self-requested a review September 22, 2022 01:05
@ANIKINKIRILL
Copy link

can we use compose multiplatform in kmm projects like in Android Studio?

neugartf added a commit to neugartf/compose-jb that referenced this pull request Oct 19, 2022
That import is not needed anymore with 1.2.0. 
JetBrains#2215
igordmn pushed a commit that referenced this pull request Oct 24, 2022
That import is not needed anymore with 1.2.0. 
#2215
@Omico Omico deleted the use-gradle-kotlin-dsl branch November 25, 2022 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants