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

Renaming project from staking_deposit to ethstaker_deposit #60

Merged
merged 2 commits into from
Jun 7, 2024

Conversation

valefar-on-discord
Copy link
Collaborator

Simple find and replace with some manual changes here and there.

Fixes #15

@remyroy
Copy link
Member

remyroy commented Jun 6, 2024

Can you run the ci-build workflow in the Actions tab on your personal repo and this branch (project-rename)? I mostly want to make sure this PR isn't breaking that part. @valefar-on-discord

Copy link
Member

@remyroy remyroy left a comment

Choose a reason for hiding this comment

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

Some very minor stuff with the Docker image name or namespace.

README.md Outdated
@@ -382,19 +380,19 @@ make build_docker
Run the following command to enter the interactive CLI:

```sh
docker run -it --rm -v $(pwd)/validator_keys:/app/validator_keys ethereum/staking-deposit-cli
docker run -it --rm -v $(pwd)/validator_keys:/app/validator_keys ethereum/ethstaker-deposit-cli
Copy link
Member

Choose a reason for hiding this comment

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

I think the image name needs to be eth-educators/ethstaker-deposit-cli (from the new DOCKER_IMAGE in Makefile). This will need more tests from #61 .

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good catch

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ack

README.md Outdated
```

You can also run the tool with optional arguments:

```sh
docker run -it --rm -v $(pwd)/validator_keys:/app/validator_keys ethereum/staking-deposit-cli new-mnemonic --num_validators=<NUM_VALIDATORS> --mnemonic_language=english --folder=<YOUR_FOLDER_PATH>
docker run -it --rm -v $(pwd)/validator_keys:/app/validator_keys ethereum/ethstaker-deposit-cli new-mnemonic --num_validators=<NUM_VALIDATORS> --mnemonic_language=english --folder=<YOUR_FOLDER_PATH>
Copy link
Member

Choose a reason for hiding this comment

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

I think the image name needs to be eth-educators/ethstaker-deposit-cli (from the new DOCKER_IMAGE in Makefile). This will need more tests from #61 .

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ack

README.md Outdated
```

Example for 1 validator on the [Prater testnet](https://prater.launchpad.ethereum.org/) using english:

```sh
docker run -it --rm -v $(pwd)/validator_keys:/app/validator_keys ethereum/staking-deposit-cli new-mnemonic --num_validators=1 --mnemonic_language=english --chain=prater
docker run -it --rm -v $(pwd)/validator_keys:/app/validator_keys ethereum/ethstaker-deposit-cli new-mnemonic --num_validators=1 --mnemonic_language=english --chain=prater
Copy link
Member

Choose a reason for hiding this comment

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

I think the image name needs to be eth-educators/ethstaker-deposit-cli (from the new DOCKER_IMAGE in Makefile). This will need more tests from #61 .

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ack

@valefar-on-discord
Copy link
Collaborator Author

Can you run the ci-build workflow in the Actions tab on your personal repo and this branch (project-rename)? I mostly want to make sure this PR isn't breaking that part. @valefar-on-discord

Ran as suggested: https://github.com/valefar-on-discord/ethstaker-deposit-cli/actions/runs/9408783246

@remyroy remyroy merged commit 42b216b into eth-educators:main Jun 7, 2024
15 checks passed
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.

Rename the staking_deposit package
2 participants