Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Can SQLDelight show up in new directory dialog? #3215

Closed
JakeWharton opened this issue May 9, 2022 · 5 comments · Fixed by #3976
Closed

Can SQLDelight show up in new directory dialog? #3215

JakeWharton opened this issue May 9, 2022 · 5 comments · Fixed by #3976

Comments

@JakeWharton
Copy link
Member

Description

Screen Shot 2022-05-08 at 11 33 01 PM

@hfhbd
Copy link
Collaborator

hfhbd commented Oct 19, 2022

Don't know if it is a bug but I am able to add it with this code:

project.plugins.withId("org.jetbrains.kotlin.jvm") {
  val srcSet = project.extensions.findByType(SourceSetContainer::class.java)!!.getByName("main")
  val kotlin = srcSet.extensions.getByName("kotlin") as SourceDirectorySet
  kotlin.srcDir(outputFolder.dir("kotlin"))
}

Bildschirm­foto 2022-10-19 um 20 35 37

@JakeWharton
Copy link
Member Author

I don't see it in your list, but adding it to the main source set isn't really what I had in mind as a solution here since that feels somewhat more like a hack.

@hfhbd
Copy link
Collaborator

hfhbd commented Oct 19, 2022

I mean the first line, it comes from my compiler and its gradle plugin. I was just surprised my custom folder is visible in the dialog. Are you sure it is a hack/bug and not a feature?

@JakeWharton
Copy link
Member Author

Showing the list of all source set directories is a feature, sure. Using that to get an unrelated folder to show up in the list seems like a hack. Our directories are not Kotlin source folders.

@JakeWharton
Copy link
Member Author

How does the Kotlin IntelliJ plugin populate this dialog with the Kotlin source sets? That's the mechanism that we need to hook into from our IDE plugin.

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

Successfully merging a pull request may close this issue.

3 participants