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

replace goerli with sepolia in the readme.md #29780

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cocoyeal
Copy link
Contributor

Since goerli testnet has been deprecated for one month, the "goerli" command in readme.md no longer works. so i replaced it with "sepolia".

@fjl
Copy link
Contributor

fjl commented May 21, 2024

We need to make a bigger change there, because simply running geth like this won't work. Sepolia also requires a beacon client to run.

@cocoyeal
Copy link
Contributor Author

@fjl Hi sir, after doing some investigation, i found that the holesky will replace goerli as a staking, infrastructure and protocol-developer testnet. should we use holesky here instead of sepolia?

@fjl
Copy link
Contributor

fjl commented May 29, 2024

The main problem is that running Geth without a CL or at least a beacon chain endpoint is not possible anymore. I personally don't care if we put Sepolia or Holesky, it'd be more important to come up with new text that explains how to connect to the testnet with a CL as well.

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.

None yet

2 participants