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
Editorial #656
Editorial #656
Conversation
Some of these editorial comments I find less clear than the original. The code changes and varint are fine. Could you please split this into two more atomic PRs so we can discuss separately? |
Feel free to leave comments on the editorial portion of this PR to keep things moving |
Co-authored-by: Theophile Wallez <theophile@wallez.net>
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.
Couple of suggestions, generally looks good.
Co-authored-by: Richard Barnes <rlb@ipv.sx>
No description provided.