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

Bump alloy [provider, rpc-types, serde, transport, rpc-client, transp… #4

Merged
merged 3 commits into from
Jul 13, 2024

Conversation

m1stoyanov
Copy link
Contributor

…ort-http] to 0.1.4, alloy-primitives to 0.7.7 and revm to 11.0.0

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

…ort-http] to 0.1.4, alloy-primitives to 0.7.7 and revm to 11.0.0
Cargo.toml Outdated Show resolved Hide resolved
Cargo.toml Outdated
Comment on lines 28 to 31
alloy-provider = {version = "0.1.4", default-features = false}
alloy-rpc-types = {version = "0.1.4", features = ["eth"]}
alloy-serde = {version = "0.1.4", default-features = false}
alloy-transport = {version = "0.1.4", default-features = false}
Copy link
Member

Choose a reason for hiding this comment

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

I'm being very pedantic here but please keep the surrounding white space i {}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed all the white spaces my Code ate :)

Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

last nit

src/backend.rs Outdated
Comment on lines 691 to 693
if number > u64::MAX {
return Ok(KECCAK_EMPTY);
}
Copy link
Member

Choose a reason for hiding this comment

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

this check is now redundant and can be removed entirely

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Completely missed that
Fixed

@mattsse mattsse merged commit 3f7b33f into foundry-rs:main Jul 13, 2024
14 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants