Skip to content
This repository has been archived by the owner on Jun 27, 2020. It is now read-only.

Missing Gradle plugin ID inhibits usage in kotlin gradle scripts #143

Open
sambsnyd opened this issue Mar 12, 2019 · 1 comment
Open

Missing Gradle plugin ID inhibits usage in kotlin gradle scripts #143

sambsnyd opened this issue Mar 12, 2019 · 1 comment

Comments

@sambsnyd
Copy link

The kotlin-frontend plugin doesn't have a Gradle plugin id.
See Gradle's guide for including a plugin id.

I'm happy to provide a pull request that adds such an id. How does "org.jetbrains.kotlin.js.frontend" sound?

There are a number of unfortunate consequences of not having this id. Chiefly, it makes this plugin much harder to use with Gradle 5.0+ and .gradle.kts build scripts.

In Gradle 4.0, handy type safe accessors like DependencyHandler.compile() could be manually generated. As of Gradle 5.0 this is no longer an option. Only plugins applied via plugins get the nice, type safe DSL accessors. Plugins require an id to be applied via plugins. So the lack of such an ID makes it huge pain to use the kotlin-frontend plugin with kotlin build scripts.

buildscript {
   // configure repositories to include kotlin-eap 
}
apply plugin: "kotlin2js"
apply plugin: // kotlin frontend 
dependencies {
    compile("group:module:version") // script compilation will fail in Gradle 5.0
}

kotlinFrontend { // script compilation will fail in Gradle 5.0

}
@markjfisher
Copy link

This would be a really nice thing to have, particularly as this whole plugin is being used in the context of kotlin.

The first thing I looked for was how to include it via an ID, and then subsequently found this issue.

Would it progress quicker if a PR was generated rather than waiting for someone to say "go ahead"?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants