Hey <@U0CHHN4F4> I’m wondering if there are any be...
# scripting
e
Hey @ilya.chernikov I’m wondering if there are any beginner issues in kotlin scripting that I can pick up. Tryna see if I can contribute back to Kotlin. Maybe something you havent gotten around to. Been reading kotlin script source for a while now, I kind of know my way around the API. Let me know if you think I can help!
i
Thank you very much @efemoney, I haven’t anticipated this, so I do not have anything prepared. But I think this is a very nice opportunity, so I’ll try to come up with something.
e
Awesome! I'll keep an eye on the issue tracker in the meantime.
i
Hi @efemoney I skimmed through the tasks I have in mind for the near future, and I see some possible improvements in the
kotlin-main-kts
that it will be very nice to have. For example: - add support for the
@CompilerOptions
annotation - add autoimport option to the
@DependsOn
annotation, that would, if asked to do so, scan the requested library and add implicit import e.g. for classses from the top-level package. Or maybe for all packages. Do you think it might be interesting for you. If so, we’d appreciate it very much.
e
Yes those sound perfect!
Are there any tracking issues for any of those?
i
Ok, good. Then you can start to have a look at the
main-kts
source in the kotlin repo, and I’ll prepare YT issues, where we can continue to discuss it.
👍 1
🎉 1
e
Perfect, thanks!
i
I should thank you for the help. 🙂
Here is the first one, let’s start with it, if you dont’ mind - https://youtrack.jetbrains.com/issue/KT-34274
👍 1
e
Hey @ilya.chernikov, I have some issues after cloning Kotlin. If you’re free, it would be great if you could give me some pointers to resolving it. https://kotlinlang.slack.com/archives/C0BUHC9HD/p1571040615004900
i
Thank you very much! I’ll try to review it in the next few days.
Please, tell me if you need an issue describing the next possible thing to work on.
e
Yes I do, that would be amazing.
i
@efemoney haven’t you forgot to commit
compile-java6.main.kts
file? Seems missing.
And please replace the line “Resolves #KT-34274” with “#KT-34274 fixed”, then our CI will update the issue after the successful build automatically.
Otherwhise it looks fine, I’m planning to merge it after the changes and the test build.
e
Yikes, will update the pr in the next couple minutes.
Updated @ilya.chernikov
i
@efemoney, thank you very much for the contribution! I merged it into master. Unfortunately I have very little time these days, so wasn’t able to prepare the next issue for you. I’ll try to do it in the next few days.
e
Thank you for merging! I’ll keep looking at the
Tools. Scripts
tag in the meantime to see if I can pick up anything 🙏