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

feat(cast run): add --etherscan-api-key to resolve contract names #9295

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

grandizzy
Copy link
Collaborator

@grandizzy grandizzy commented Nov 11, 2024

Motivation

Closes #3473

Solution

  • fix get_remote_chain_id: do not assume mainnet chain if mainnet in RPC URL (Alchemy has opt-mainnet, arb-mainnet etc.)
  • add --etherscan-api-key for cast run
  • test

crates/cast/bin/cmd/run.rs Outdated Show resolved Hide resolved
@grandizzy grandizzy force-pushed the issue-3473 branch 2 times, most recently from 518e7f4 to 1696273 Compare November 11, 2024 13:04
@grandizzy grandizzy marked this pull request as draft November 11, 2024 13:27
@grandizzy grandizzy changed the title feat(cast run): add --etherscan-api-key flag to resolve contract names feat(cast run): add --etherscan-api-key to resolve contract names Nov 11, 2024
@grandizzy grandizzy requested a review from klkvr November 11, 2024 14:16
@grandizzy grandizzy marked this pull request as ready for review November 11, 2024 14:16
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.

lgtm

@mattsse mattsse merged commit 8c01706 into foundry-rs:master Nov 11, 2024
21 checks passed
@grandizzy grandizzy deleted the issue-3473 branch November 11, 2024 14:49
rplusq pushed a commit to rplusq/foundry that referenced this pull request Nov 29, 2024
foundry-rs#9295)

* fix(cast run): decode traces for non mainnet

* Add test

* Changes after review: use EtherscanOpts, remove short -e from evm_version
Simplify test to avoid rate limiting.
@grandizzy grandizzy added T-feature Type: feature C-cast Command: cast labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cast Command: cast T-feature Type: feature
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

feat(cast): add --etherscan-api-key flag to resolve contract names
4 participants