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

Fixed typo in stdout log #5317

Merged
merged 1 commit into from
Apr 6, 2020
Merged

Conversation

chrishobcroft
Copy link
Contributor

"Round robing sync request failed"

changed to

"Round robin sync request failed"

TODO(you): choose "part of" or "resolves" and the associated github issue #.

[Part of|Resolves] #531


Description

Write why you are making the changes in this pull request

Write a summary of the changes you are making

Link anything that would be helpful or relevant to the reviewers

"Round robin**g** sync request failed"

changed to

"Round robin sync request failed"
@codecov
Copy link

codecov bot commented Apr 6, 2020

Codecov Report

Merging #5317 into master will decrease coverage by 4.28%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master   #5317      +/-   ##
=========================================
- Coverage    8.99%   4.70%   -4.29%     
=========================================
  Files         261     230      -31     
  Lines       21204   19289    -1915     
=========================================
- Hits         1907     908     -999     
+ Misses      18983   18239     -744     
+ Partials      314     142     -172     

@chrishobcroft
Copy link
Contributor Author

Hey @nisdas what checks does this need to pass?

@nisdas
Copy link
Member

nisdas commented Apr 6, 2020

Hey @chrishobcroft , it looks good to me. Sorry forgot to merge it in, thanks for your contribution !

@nisdas nisdas merged commit b7afc90 into prysmaticlabs:master Apr 6, 2020
@criticaltv
Copy link

@chrishobcroft you're an Ethereum Core Dev now ;)

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