Skip to content
This repository has been archived by the owner on Mar 5, 2024. It is now read-only.

Schedule shapella for chiado #34

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Schedule shapella for chiado #34

wants to merge 3 commits into from

Conversation

dapplion
Copy link
Contributor

@nflaig
Copy link

nflaig commented May 17, 2023

We have chiado config in Lodestar, we probably don't want to keep maintaining it here

What is the general plan regarding this package? It is really out of date and there is no official release yet

@dapplion
Copy link
Contributor Author

What is the general plan regarding this package? It is really out of date and there is no official release yet

@pablomendezroyo ?

We have chiado config in Lodestar, we probably don't want to keep maintaining it here

Updated PR to use --network chiado

Copy link

@nflaig nflaig left a comment

Choose a reason for hiding this comment

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

If we decide to make this package production ready, we can just close this PR and do a full refactor in another PR.

Copy link

Choose a reason for hiding this comment

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

Just noticed, we can probably also remove --genesisStateFile and --bootnodes.

Same update also needs to be applied to validator/entrypoint.sh

@pablomendezroyo
Copy link

pablomendezroyo commented Jun 20, 2023

What is the general plan regarding this package? It is really out of date and there is no official release yet

AFAIK there is no plan to publish this pkg, it was mainly created for testing purposes.

Copy link

@nflaig nflaig left a comment

Choose a reason for hiding this comment

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

Docker build is broken now

COPY chiado/config.yaml /usr/config.yaml
COPY chiado/genesis.ssz /usr/genesis.ssz

but considering this package is broken in many differente places let's merge this as is

@dapplion
Copy link
Contributor Author

@pablomendezroyo can you add CI?

@pablomendezroyo
Copy link

yes!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants