-
Notifications
You must be signed in to change notification settings - Fork 543
[wasmJs] Enable wasmJs target #5531
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of naming suggestions (and some files will need to be moved), but otherwise this looks good. Thank you!
moduleKind.set(JsModuleKind.MODULE_UMD) | ||
} | ||
} | ||
|
||
applyDefaultHierarchyTemplate { | ||
common { | ||
group("jsCommon") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
group("jsCommon") { | |
group("web") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
runtime/build.gradle
Outdated
@@ -16,6 +16,16 @@ apiValidation { | |||
} | |||
|
|||
kotlin { | |||
applyDefaultHierarchyTemplate { | |||
it.common { | |||
it.group("jsNativeCommon") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it.group("jsNativeCommon") { | |
it.group("nonJvm") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
938bfa7
to
6713b83
Compare
Great to see this land! Amazing work Ilya. When can we expect a release with the Wasm support? |
@skaldebane this is only 1st PR of 3 🙂 |
Is there a snapshot available with this merged PR? I cannot find the latest snapshots anywhere. |
@Thomas-Vos here's a snapshot by @ahna92 in my another PR but you will need PAT to access it #5534 (comment) |
Enable wasmJs target
This PR adds support for the WebAssembly JavaScript (wasmJs) target in SQLDelight.
Changes
Build Configuration
wasmJs()
target alongside existingjs()
target in multiplatform conventionsjsCommon
group that contains both JS and WasmJS targetsSource Reorganization
CurrentThreadId.kt
in:runtime
fromjsMain
tojsCommonMain
to share between JS and WasmJSCloseable.kt
fromjsMain
andnativeMain
intojsNativeCommonMain
to share between JS, WasmJS, and Native targets.