Skip to content

Conversation

aSemy
Copy link
Contributor

@aSemy aSemy commented Jun 9, 2024

  • The kotlin-dsl plugin also applies java-gradle-plugin, so applying both plugins is unnecessary
  • Gradle Plugins should use the version of Kotlin embedded into Gradle. Applying kotlin("jvm") can cause issues, as it will use a non-embedded Kotlin, and configure some defaults that might cause issues (for example, adding an API dependency on kotlin-stdlib, and this might clash with other plugins)

- The `kotlin-dsl` plugin also applies `java-gradle-plugin`, so applying both plugins is unnecessary
- Gradle Plugins should use the version of Kotlin embedded into Gradle. Applying `kotlin("jvm")` can cause issues, as it will use a non-embedded Kotlin, and configure some defaults that might cause issues (for example, adding an API dependency on kotlin-stdlib, and this might clash with other plugins)
@aSemy aSemy marked this pull request as draft June 9, 2024 08:34
@aSemy aSemy marked this pull request as ready for review June 9, 2024 08:38
@Kantis Kantis added this pull request to the merge queue Jun 9, 2024
Merged via the queue into kotest:master with commit 1713eb7 Jun 9, 2024
@aSemy aSemy deleted the fix/kotest-gradle-plugin-update-build-plugins branch June 9, 2024 22:07
@aSemy aSemy added the builds 🐘 PRs / Issues related to the CI/CD pipelines or Gradle builds label Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builds 🐘 PRs / Issues related to the CI/CD pipelines or Gradle builds
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants