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

doc: keep variable name consistent in README #79

Merged
merged 2 commits into from Nov 28, 2019

Conversation

@HAOYUatHZ
Copy link
Contributor

HAOYUatHZ commented Nov 28, 2019

Currently p2pCtx variable name in README is inconsistent,
namely ctx in L55 but p2pCtx in L56.

We'd better keep it consistent for better understanding.

HAOYUatHZ added 2 commits Nov 28, 2019
Currently p2pCtx variable name in README is inconsistent,
namely `ctx` in L55 but `p2pCtx` in L56.

We'd better keep it consistent for better understanding.
@notatestuser notatestuser self-requested a review Nov 28, 2019
@notatestuser notatestuser requested review from ackratos and rubenillodo Nov 28, 2019
@notatestuser

This comment has been minimized.

Copy link
Member

notatestuser commented Nov 28, 2019

Thanks!

@notatestuser notatestuser merged commit 883f207 into binance-chain:master Nov 28, 2019
1 check passed
1 check passed
Test
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.