-
Notifications
You must be signed in to change notification settings - Fork 25
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
feat(core): 🎸 compile wasm #543
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #543 +/- ##
==========================================
- Coverage 85.27% 85.19% -0.09%
==========================================
Files 179 179
Lines 25337 25378 +41
==========================================
+ Hits 21607 21621 +14
- Misses 3730 3757 +27 ☔ View full report in Codecov by Sentry. |
d69ac35
to
fdaa5f0
Compare
tests/Cargo.toml
Outdated
@@ -18,6 +18,7 @@ paste.workspace = true | |||
ribir = {path = "../ribir", features = ["material", "widgets"]} | |||
ribir_dev_helper = {path = "../dev-helper"} | |||
ribir_geom = {path = "../geom"} | |||
ribir_core = {path = "../core"} |
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.
Unnecessary import
d3093ce
to
69428ea
Compare
.github/workflows/ci.yml
Outdated
@@ -59,4 +59,18 @@ jobs: | |||
{ | |||
echo ./README.md | |||
find "./docs" -name "*.md" | |||
} | xargs -I {} rustdoc --test {} -L target/debug/deps/ --edition 2018 --extern ribir=target/debug/libribir.rlib | |||
} | xargs -I {} rustdoc -Z unstable-options --test --no-run {} -L target/debug/deps/ --edition 2018 --extern ribir=target/debug/libribir.rlib |
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.
should keep not use unstable-options to test docs
9825af7
to
45760c9
Compare
fd473a8
to
9fe4714
Compare
76f382e
to
321df76
Compare
Purpose of this Pull Request
compile wasm, relative to #542
Checklist Before Merging
Please ensure the following are completed before merging:
CHANGELOG.md
file.Breaking
section of theCHANGELOG.md
file.Additional Information
If you're unsure about which branch to submit your Pull Request to, or when it will be released after being merged, please refer to our Release Guide.
If you're working on a widget and need help writing test cases, we have some macros that can assist you. Please refer to the Ribir Dev Helper documentation.