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

Kotlin support #2177

Closed
Zoweb opened this issue Oct 20, 2018 · 3 comments · Fixed by #2210
Closed

Kotlin support #2177

Zoweb opened this issue Oct 20, 2018 · 3 comments · Fixed by #2210

Comments

@Zoweb
Copy link
Contributor

Zoweb commented Oct 20, 2018

🙋 feature request

Support for building and including Kotlin (.kt) files. As Kotlin is getting increasingly popular, it makes sense for support to be built-in.

🤔 Expected Behavior

When a .kt file is found, it should be included in the output as a JavaScript file or a WASM file (would be configurable)

😯 Current Behavior

The .kt file is just copied into the output directory.

💁 Possible Solution

Node packages exist that should be able to build Kotlin files, however after testing kotlin-node could not be installed and node-kotlin would generate an empty zip inside a zip.

Instead kotlinc could be downloaded from JetBrains/kotlin and run with the correct arguments to generate .js files, which could then be inputted back into the system to create the output.

🔦 Context

I'm trying to use Kotlin instead of JavaScript or TypeScript because I prefer the language and how it works. As Kotlin can build to both JavaScript and WASM, it can be used as the single programming language in the project, as well as a server system.

💻 Examples

N/A

@Zoweb
Copy link
Contributor Author

Zoweb commented Oct 27, 2018

It looks like WebPack has support for Kotlin, so maybe it could be ported over to Parcel.

@DeMoorJasper
Copy link
Member

Feel free to try this and open up a pr

Sent with GitHawk

@Zoweb Zoweb mentioned this issue Oct 28, 2018
3 tasks
@Zoweb
Copy link
Contributor Author

Zoweb commented Oct 28, 2018

OK, I've created the pr - #2210

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.

2 participants