Skip to content

Conversation

@sainoe
Copy link
Collaborator

@sainoe sainoe commented Dec 3, 2025

This PR fixes some typos and addresses other observations found while reviewing the docs and running a local testnet.

-d '{"jsonrpc":"2.0","method":"eth_getBalance","params":["0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266","latest"],"id":1}'
```

**Send a transaction:**
Copy link
Collaborator Author

@sainoe sainoe Dec 3, 2025

Choose a reason for hiding this comment

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

Doesn't work in the testnet setup, as it requires the sender's private keys to be available in the EVM client.

@sainoe sainoe marked this pull request as ready for review December 3, 2025 16:48
</details>

Running this command should results in the following output:
Running this command should result in the following output:
Copy link
Contributor

Choose a reason for hiding this comment

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

At least I was consistent :)

**Verify installations:**
```bash
rustc --version # Should show rustc 1.85+
rustc --version # Should show rustc 1.88+
Copy link
Contributor

Choose a reason for hiding this comment

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

Which crates requires 1.88+?
I am able to verify MSRV 1.85 for:

  • app
  • cli
  • engine
  • utils
  • types

We should also update https://github.com/informalsystems/emerald/blob/main/Cargo.toml#L18 accordingly

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh and I see we are missing rust-version = { workspace = true } here https://github.com/informalsystems/emerald/blob/main/cli/Cargo.toml#L8

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Alloy and Reth require 1.88+.

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.

4 participants