-
Notifications
You must be signed in to change notification settings - Fork 232
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
refactor: refactor peer store #2964
Conversation
78159f9
to
3493083
Compare
9f9b698
to
a2a5601
Compare
72004f5
to
322209f
Compare
322209f
to
e34ddd9
Compare
e34ddd9
to
fca5c4d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have read all related issues (I found two).
Two participants disappeared after several comments, I'm not sure they agreed or disagreed this PR.
I don't know whether our team have reached a consensus with this PR.
I only approve that the code is same as the issue described.
bors r=quake,yangby-cryptape |
Build succeeded: |
What problem does this PR solve?
What is changed and how it works?
Check List
Tests
Release note