-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
chore(deps): bump all dependencies, revm 7 #7365
Conversation
```text Updating crates.io index Updating git repository `https://github.com/alloy-rs/alloy` Updating git repository `https://github.com/paradigmxyz/evm-inspectors` Updating ahash v0.8.10 -> v0.8.11 Updating bumpalo v3.15.3 -> v3.15.4 Updating cc v1.0.88 -> v1.0.90 Adding cfg_aliases v0.1.1 (latest: v0.2.0) Updating chrono v0.4.34 -> v0.4.35 Updating clap v4.5.1 -> v4.5.2 Updating clap_builder v4.5.1 -> v4.5.2 Updating const-hex v1.11.1 -> v1.11.2 Updating ctrlc v3.4.2 -> v3.4.4 Updating enr v0.10.0 -> v0.10.1 Updating env_logger v0.11.2 -> v0.11.3 Updating foundry-compilers v0.3.9 -> v0.3.10 Updating http v0.2.11 -> v0.2.12 (latest: v1.1.0) Adding jobserver v0.1.28 Updating js-sys v0.3.68 -> v0.3.69 Updating nix v0.27.1 -> v0.28.0 Updating pin-project v1.1.4 -> v1.1.5 Updating pin-project-internal v1.1.4 -> v1.1.5 Adding proc-macro-crate v3.1.0 Updating regex-automata v0.4.5 -> v0.4.6 Updating reqwest v0.11.24 -> v0.11.25 Updating serde_path_to_error v0.1.15 -> v0.1.16 Updating strum v0.26.1 -> v0.26.2 Updating strum_macros v0.26.1 -> v0.26.2 Updating system-configuration v0.5.1 -> v0.6.0 Updating system-configuration-sys v0.5.0 -> v0.6.0 Updating wasm-bindgen v0.2.91 -> v0.2.92 Updating wasm-bindgen-backend v0.2.91 -> v0.2.92 Updating wasm-bindgen-futures v0.4.41 -> v0.4.42 Updating wasm-bindgen-macro v0.2.91 -> v0.2.92 Updating wasm-bindgen-macro-support v0.2.91 -> v0.2.92 Updating wasm-bindgen-shared v0.2.91 -> v0.2.92 Updating web-sys v0.3.68 -> v0.3.69 Updating yansi v1.0.0-rc.1 -> v1.0.0 ```
pub(crate) ecx: &'b mut EvmContext<DB>, | ||
pub(crate) ecx: &'evm mut InnerEvmContext<DB>, | ||
/// The precompiles context. | ||
pub(crate) precompiles: &'evm mut ContextPrecompiles<DB>, |
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.
I've inlined EvmContext
because otherwise it invokes the deref on self.ecx which doesn't allow multiple mutable borrows (e.g. ecx.db and ecx.journaled_state would be in conflict)
same reason for let ecx = &mut ecx.inner;
Note for future: rustls/openssl will have to be enabled somewhere when ethers is removed |
Flaky test I guess? It uses ethers providers so I don't think this is related |
Supersedes #7356