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

Updated documentation regarding starting a beacon chain #3557

Merged
merged 1 commit into from
Dec 29, 2021

Conversation

dadepo
Copy link
Contributor

@dadepo dadepo commented Dec 29, 2021

Motivation

Some portion of the documentation has since diverged from the current behaviour of the cli.
This PR updates this outdated part of the documentation.

Description

Updated the command listed in the documentation to reflect the fact that init is no longer a sub command of beacon. Also provided more instructions regarding the location of an eth1 client that the beacon node expects.

@CLAassistant
Copy link

CLAassistant commented Dec 29, 2021

CLA assistant check
All committers have signed the CLA.

@codeclimate
Copy link

codeclimate bot commented Dec 29, 2021

Code Climate has analyzed commit 489a655 and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Contributor

@dapplion dapplion left a comment

Choose a reason for hiding this comment

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

Thanks @dadepo and welcome to the repo! ❤️

@codecov
Copy link

codecov bot commented Dec 29, 2021

Codecov Report

Merging #3557 (489a655) into master (fcbc459) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3557   +/-   ##
=======================================
  Coverage   37.56%   37.56%           
=======================================
  Files         311      311           
  Lines        8272     8272           
  Branches     1278     1278           
=======================================
  Hits         3107     3107           
  Misses       5017     5017           
  Partials      148      148           

@dapplion dapplion merged commit 240085f into ChainSafe:master Dec 29, 2021
@dadepo dadepo deleted the patch-1 branch December 30, 2021 17:53
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

3 participants