You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We now support kotlinlib built in to Mill, so we should support Kotlin modules via IntelliJ BSP as well. AFAIK the Kotlin community is mostly on IntelliJ, so we don't need to worry about VSCode for now. And since kotlinlib is now bundled with Mill, we don't need to worry about making the BSP server extendable (#2598), and can just hard-code support for KotlinModule for now
I'm not sure what exactly will need to be done to make Mill/BSP/IntelliJ/Kotlin work. It seems we have MillJavaBuildServer.scala and MillScalaBuildServer.scala, so likely we will need a corresponding MillKotlinBuildServer.scala file. We may also need changes on the IntelliJ side to allow the IntelliJ BSP client to generate Kotlin modules, if it doesn't already support that
The text was updated successfully, but these errors were encountered:
Sure, I was actually in the midst of adding basic BSP integration tests (that would have caught BSP issues in the current state of #3599, and are needed for future BSP developments, I think)
Splitting this out of #3397
We now support
kotlinlib
built in to Mill, so we should support Kotlin modules via IntelliJ BSP as well. AFAIK the Kotlin community is mostly on IntelliJ, so we don't need to worry about VSCode for now. And sincekotlinlib
is now bundled with Mill, we don't need to worry about making the BSP server extendable (#2598), and can just hard-code support forKotlinModule
for nowI'm not sure what exactly will need to be done to make Mill/BSP/IntelliJ/Kotlin work. It seems we have
MillJavaBuildServer.scala
andMillScalaBuildServer.scala
, so likely we will need a correspondingMillKotlinBuildServer.scala
file. We may also need changes on the IntelliJ side to allow the IntelliJ BSP client to generate Kotlin modules, if it doesn't already support thatThe text was updated successfully, but these errors were encountered: