Skip to content

Commit

Permalink
fix: review suggestions
Browse files Browse the repository at this point in the history
  • Loading branch information
danil-pavlov committed Oct 30, 2023
1 parent 6579b63 commit f47f597
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions docs/topics/multiplatform/multiplatform-compatibility-guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -326,10 +326,11 @@ kotlin {
val myCustomIntermediateSourceSet by creating {
dependsOn(commonMain)
}

// Option #1. Shorter and more readable, use it when possible:
val jvmMain by getting { // Usually, the name of the default source set
// is a simple concatenation of the target name and the compilation name

// Option #1. Shorter and more readable, use it when possible.
// Usually, the name of the default source set
// is a simple concatenation of the target name and the compilation name:
val jvmMain by getting {
dependsOn(myCustomIntermediateSourceSet)
}

Expand Down Expand Up @@ -503,11 +504,11 @@ projects.
In Kotlin 1.9.0, a deprecation warning is introduced when the `android` name is used in Kotlin Multiplatform projects.
<anchor name="declaring-multiple-targets"></anchor>
## Declaring multiple targets of the same type
## Declaring several similar targets
**What's changed?**

We discourage declaring multiple targets of the same type in a single Gradle project. For example:
We discourage declaring several similar targets in a single Gradle project. For example:

```kotlin
kotlin {
Expand Down Expand Up @@ -632,7 +633,7 @@ the Kotlin Gradle plugin, making it easier to use and maintain the resulting bui
* 2.0: report an error in such cases, causing the build to fail

<anchor name="jvmWithJava-preset-deprecation"></anchor>
## Deprecated the jvmWithJava preset
## Deprecated jvmWithJava preset

**What's changed?**
Expand Down Expand Up @@ -731,8 +732,7 @@ removed from the public API of the Kotlin Gradle plugin in future releases. This
**What's the best practice now?**

Use respective [Kotlin targets](https://kotlinlang.org/docs/multiplatform-dsl-reference.html#targets) instead.
It's more concise and cleaner. For example:
Use respective [Kotlin targets](https://kotlinlang.org/docs/multiplatform-dsl-reference.html#targets) instead, for example:

<table header-style="top">
<tr>
Expand Down

0 comments on commit f47f597

Please sign in to comment.