Skip to content

feat: move local signers to a separate crate, fix wasm#306

Merged
DaniPopes merged 12 commits into
mainfrom
dani/signer-wallet
Mar 14, 2024
Merged

feat: move local signers to a separate crate, fix wasm#306
DaniPopes merged 12 commits into
mainfrom
dani/signer-wallet

Conversation

@DaniPopes

@DaniPopes DaniPopes commented Mar 14, 2024

Copy link
Copy Markdown
Member

Closes #262, but does not fix wasm for alloy-network. This is because k256 always imports getrandom through multiple dependencies that use rand_core.
Closes #294

Fix wasm checks for most crates, see wasm-wasi GHA job. This job should have the same results as wasm32-unknown-unknown with getrandom/js. Only crates that we cannot make compile is Trezor (libusb), GCP (idk), IPC (no fs)

Closes #74

@DaniPopes DaniPopes force-pushed the dani/signer-wallet branch from 3154b86 to c4a5737 Compare March 14, 2024 17:14

@mattsse mattsse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this makes sense to me!

use crate::{TransportError, TransportResult};
use alloy_json_rpc::ResponsePacket;
/// Future for transport-level requests.
pub type TransportFut<'a, T = alloy_json_rpc::ResponsePacket, E = TransportError> = Pbf<'a, T, E>;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unrelated changes?

@DaniPopes DaniPopes Mar 14, 2024

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasming network as much as possible, it compiles now on wasm32-wasi or if getrandom.features = ["js"]

@DaniPopes DaniPopes changed the title feat: move local signers to a separate crate feat: move local signers to a separate crate, fix wasm Mar 14, 2024
@DaniPopes DaniPopes requested a review from onbjerg March 14, 2024 19:31

@onbjerg onbjerg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@DaniPopes DaniPopes merged commit 42594c2 into main Mar 14, 2024
@DaniPopes DaniPopes deleted the dani/signer-wallet branch March 14, 2024 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants