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

Move libp2p to a fork and enable CI wasm checks #1294

Merged
merged 2 commits into from Jan 26, 2023
Merged

Conversation

jsdanielh
Copy link
Contributor

  • Move the libp2p dependency to a fork to enable compilation for wasm and non-wasm targets.
    This is due to the fact that the gossipsub protocol got the wasm support dropped in this PR.
  • Add a job in the build+test Github Actions workflow for compiling to wasm and generating the JS bindings for the web-client

Pull request checklist

  • All tests pass. The project builds and runs.
  • I have resolved any merge conflicts.
  • I have resolved all clippy and rustfmt warnings.

@jsdanielh jsdanielh added the WIP This pull request is still work in progress label Jan 26, 2023
Move the `libp2p` dependency to a fork to enable compilation for
`wasm` and non-`wasm` targets.
This is due to the fact that the `gossipsub` protocol got the `wasm`
support dropped in this
[PR](libp2p/rust-libp2p#2506).
Add a job in the `build+test` Github Actions workflow for compiling
to `wasm` and generating the JS bindings for the `web-client`
@codecov
Copy link

codecov bot commented Jan 26, 2023

Codecov Report

Base: 67.55% // Head: 67.54% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (1a50e74) compared to base (6ffd680).
Patch has no changes to coverable lines.

Additional details and impacted files
@@              Coverage Diff              @@
##           albatross    #1294      +/-   ##
=============================================
- Coverage      67.55%   67.54%   -0.01%     
=============================================
  Files            397      397              
  Lines          50818    50818              
=============================================
- Hits           34331    34327       -4     
- Misses         16487    16491       +4     
Flag Coverage Δ
unittests 67.54% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
handel/src/todo.rs 84.12% <0.00%> (-3.18%) ⬇️
handel/src/aggregation.rs 91.15% <0.00%> (-0.30%) ⬇️
network-libp2p/src/network.rs 74.64% <0.00%> (+0.07%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jsdanielh jsdanielh removed the WIP This pull request is still work in progress label Jan 26, 2023
@jsdanielh jsdanielh merged commit 1a50e74 into albatross Jan 26, 2023
@jsdanielh jsdanielh deleted the jsdanielh/wasm3 branch January 26, 2023 22:55
@jsdanielh jsdanielh added this to the Nimiq 2.0 Testnet milestone Jan 31, 2023
@sisou sisou added the WASM label Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants