Skip to content

chore: clearer error message when function signature does not contain parentheses #9478

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

yipu3
Copy link

@yipu3 yipu3 commented Dec 4, 2024

Motivation

Closes #9479
When I wrongly input a function signature without parentheses:

./target/debug/cast call 0x7a250d5630B4cF539739dF2C5dAcb4c659F2488D 'deposit'

The error message is a bit of confusing:

Error: If you wish to fetch function data from EtherScan, please provide an API key.

Which costs me some time to figure out what's the point.

A clearer error message is needed.

Solution

Change the message into:

Function signature does not contain parentheses. If you wish to fetch function data from Etherscan, please provide an API key.

DaniPopes
DaniPopes previously approved these changes Dec 4, 2024
@DaniPopes DaniPopes enabled auto-merge (squash) December 4, 2024 07:50
@DaniPopes
Copy link
Member

Thanks!

@yipu3 yipu3 requested a review from yash-atreya December 4, 2024 08:48
@zerosnacks zerosnacks changed the title chore: clearer error message when function signature does not contain… chore: clearer error message when function signature does not contain parentheses Dec 4, 2024
@zerosnacks zerosnacks self-requested a review December 4, 2024 09:10
@zerosnacks zerosnacks disabled auto-merge December 4, 2024 09:45
@DaniPopes DaniPopes self-requested a review December 5, 2024 18:58
@jenpaff jenpaff moved this to Ready For Review in Foundry Apr 15, 2025
@DaniPopes DaniPopes enabled auto-merge (squash) April 29, 2025 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready For Review
Development

Successfully merging this pull request may close these issues.

chore: clearer error message when function signature does not contain parentheses
5 participants