-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add sepolia #2639
Add sepolia #2639
Conversation
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.
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.
Just left a small comment on updating an exception message but lgtm otherwise 👍🏼
@@ -15,4 +15,5 @@ | |||
3: "ropsten", | |||
4: "rinkeby", | |||
5: "goerli", | |||
11155111: "sepolia", |
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.
Since this list is fairly small and we removed Kovan, it would be nice to update this exception message to include 11155111 (sepolia)
as well:
web3.py/ethpm/validation/uri.py
Lines 105 to 107 in 845bce4
f"Chain ID: {chain_id} is not supported. Supported chain ids include: " | |
"1 (mainnet), 3 (ropsten), 4 (rinkeby), and 5 (goerli)" | |
"Please try again with a valid registry URI." |
* Add Sepolia testnet * Add Sepolia to ethpm validation error message
* Add Sepolia testnet * Add Sepolia to ethpm validation error message
What was wrong?
Sepolia is a new testnet that should be added to web3's auto providers.
Related to Issue #2627
How was it fixed?
Added it!
Todo:
Cute Animal Picture