vertx-lang-kotlin icon indicating copy to clipboard operation
vertx-lang-kotlin copied to clipboard

only the verticle with kt or kts suffix needs to be compiled

Open okou19900722 opened this issue 6 years ago • 7 comments

like GroovyVerticleFactory and ScalaVerticleFactory

okou19900722 avatar Nov 20 '18 04:11 okou19900722

can you add a test with a .kts file ?

vietj avatar Nov 20 '18 07:11 vietj

I am not very clear about the compilation process of kts files, because I have always used kotlin as java in use, not as a script. Maybe others can provide more help

okou19900722 avatar Nov 20 '18 07:11 okou19900722

@gmariotti can you hint here ?

vietj avatar Nov 20 '18 07:11 vietj

@vietj not really sure about the compilation process of a kts, I'll need to investigate it. @okou19900722 you can probably look at https://github.com/JetBrains/kotlin/tree/master/libraries/examples/kotlin-jsr223-local-example to figure out how to do it

gmariotti avatar Nov 20 '18 09:11 gmariotti

jsr223 is an existing feature. I wonder if the kts file will be like a kt file. When it is packaged, it will be compiled by the kotlin plugin.So if the kts file is not compiled, then it should not require a test for kts file. @vietj @gmariotti

okou19900722 avatar Nov 20 '18 09:11 okou19900722

Sorry @okou19900722 , I honestly didn't find a lot of info on kts files. However, except for the Kotlin Gradle DSL, I haven't see a lot of cases where kts was used, so I'm not really sure if it makes sense to support this format at all. What do you think @vietj?

gmariotti avatar Dec 16 '18 10:12 gmariotti

Any updates about this pr? Should we merge this pr first and deal with kts suffix/prefix later?

chengenzhao avatar Mar 18 '19 03:03 chengenzhao

Compiler not supported anymore #166

tsegismont avatar Oct 06 '23 08:10 tsegismont