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

BOLT 7: readability fixup #477

Merged
merged 13 commits into from Oct 29, 2018

Conversation

Projects
None yet
3 participants
@niftynei
Contributor

niftynei commented Sep 12, 2018

make it a bit easier to read.

fixes #443

@niftynei niftynei changed the title from routing gossip: clarify what 'redundant' means to WIP routing gossip: readability fixup Sep 12, 2018

niftynei added some commits Sep 12, 2018

routing-gossip: add clarifying statement to Initial Sync section
The Initial Sync section is hard to parse, as is, because it's presented
without context. This adds a small clarifying statement, in particular
highlighting where the mode is triggered from (init message features)
and what in particular is being synced (gossip messages)

@niftynei niftynei changed the title from WIP routing gossip: readability fixup to routing gossip: readability fixup Sep 12, 2018

@niftynei niftynei changed the title from routing gossip: readability fixup to BOLT 7: readability fixup Sep 12, 2018

@@ -540,7 +540,7 @@ An endpoint node:
### Requirements
The final node:
A message receiving node:

This comment has been minimized.

@rustyrussell

rustyrussell Sep 14, 2018

Collaborator

A receiving node?

Show resolved Hide resolved 07-routing-gossip.md Outdated
@rustyrussell

This comment has been minimized.

Collaborator

rustyrussell commented Oct 18, 2018

Assigned to @sstone for final review; I believe he had questions, otherwise @sstone please commit as per minutes:
https://docs.google.com/document/d/1GaH4ERsFxnVRceW-hG_-Ul5TOjRv40yB2S3-CEe5n9s/edit#

Show resolved Hide resolved 07-routing-gossip.md
@sstone

sstone approved these changes Oct 26, 2018

@rustyrussell rustyrussell merged commit 57dad81 into lightningnetwork:master Oct 29, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment