-
Notifications
You must be signed in to change notification settings - Fork 434
Conversation
Until a new version of `kotlin-dsl` is published.
Because the `kotlin.script` classes required by the `kotlin-dsl` were moved there.
This reverts commit 5d9d054.
To silence IntelliJ warning about it.
Broken after the latest upgrade to Gradle master.
@eskatos, this is ready for your review. There are 2 open issues:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Upgrade went not so bad 👍
I requested one change about the kotlin-eap repository.
Let's see if @lptr has some light to shed on us about the Groovy 2.5 FastString
issue with ProjectBuilder
.
Did you find an existing issue about the kapt problem?
kotlin { | ||
experimental.coroutines = Coroutines.ENABLE | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\o/
subprojects/provider/src/main/kotlin/org/gradle/kotlin/dsl/provider/KotlinScriptEvaluator.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to merge!
Let's create issues to track the ProjectBuilder
issue with Groovy 2.5 and the kapt 1.3 problem.
Now that the upgrade to Kotlin 1.3.0-rc-57 is complete.
Upgrade Kotlin to the latest EAP release
1.3-rc-57
and automatically add thekotlinEap()
repository to every project'sbuildscript.repositories
andrepositories
to ease migration.