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

Update Go-Ethereum Dependency #8788

Merged
merged 6 commits into from
Apr 20, 2021
Merged

Update Go-Ethereum Dependency #8788

merged 6 commits into from
Apr 20, 2021

Conversation

rauljordan
Copy link
Contributor

What type of PR is this?

Other

What does this PR do? Why is it needed?

This PR updates our bazel-go-ethereum dep to use the latest version, which contains the latest master branch of go-ethereum including some key additions to the eth1 <> eth2 merge specification.

@rauljordan rauljordan requested a review from a team as a code owner April 19, 2021 18:09
terencechain
terencechain previously approved these changes Apr 19, 2021
@codecov
Copy link

codecov bot commented Apr 20, 2021

Codecov Report

Merging #8788 (16d682d) into develop (70dc22a) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #8788      +/-   ##
===========================================
+ Coverage    60.49%   60.53%   +0.04%     
===========================================
  Files          519      519              
  Lines        36290    36290              
===========================================
+ Hits         21954    21969      +15     
+ Misses       11166    11150      -16     
- Partials      3170     3171       +1     

@terencechain terencechain merged commit 2634430 into develop Apr 20, 2021
@delete-merged-branch delete-merged-branch bot deleted the update-geth-dep branch April 20, 2021 13:57
@@ -57,7 +57,7 @@ func (node *Eth1Node) Start(ctx context.Context) error {
return errors.New("go-ethereum binary not found")
}

eth1Path := path.Join(e2e.TestParams.TestPath, "eth1data/")
eth1Path := path.Join(e2e.TestParams.LogPath, "eth1data/")
Copy link
Member

Choose a reason for hiding this comment

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

Why change this?

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.

3 participants