-
Notifications
You must be signed in to change notification settings - Fork 7
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
Support WebAssembly targets #111
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #111 +/- ##
============================================
+ Coverage 84.17% 84.26% +0.09%
Complexity 151 151
============================================
Files 22 22
Lines 537 534 -3
Branches 89 89
============================================
- Hits 452 450 -2
+ Misses 58 57 -1
Partials 27 27
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
||
The supported Native targets are: | ||
|
||
| Tier 1 | Tier 2 | Tier 3 | |
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.
What do you mean by tiers here?
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.
This table aligns with the tiers specified in the documentation:
https://kotlinlang.org/docs/native-target-support.html
I'd like to enable other Native targets in the future.
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.
Interesting, I missed the inception of this classification! Could you please add think link to the readme as well, something like:
| Tier 1 | Tier 2 | Tier 3 | | |
The supported Native targets are (following the Kotlin/Native [target support guidelines](https://kotlinlang.org/docs/native-target-support.html)): | |
| Tier 1 | Tier 2 | Tier 3 | |
Edit: I updated it myself to merge the PR quicker
Being WebAssembly is progressing at a fast pace, the more WASM-enabled libraries, the better.
Updating to 1.9.24 is required for using Node.js 22.