Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

* exclude showing of KotlinWithLibraryConfigurators in Kobalt based p… #1067

Closed
wants to merge 1 commit into from

Conversation

dmitry-zhuravlev
Copy link
Member

Kobalt have its own Kotlin libs configurator: https://github.com/cbeust/kobalt-intellij-plugin/blob/master/src/com/beust/kobalt/intellij/configurator/KotlinKobaltProjectConfigurator.kt
Doesn't need to show up Java/JS default configurators to not confuse users.

@goodwinnk
Copy link
Contributor

It looks with the change in this line

return module.getBuildSystemType() == BuildSystemType.JPS
, this is all that left from this pull request e36b103.

@dmitry-zhuravlev Can you please confirm that it will be enough?

@goodwinnk goodwinnk assigned goodwinnk and unassigned Alefas Dec 3, 2018
@goodwinnk
Copy link
Contributor

I have pushed the commit to master: ab259c2.

Thank you for the contribution.

@goodwinnk goodwinnk closed this Dec 4, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants