Skip to content

Fixed typos#306

Closed
dimitris-t wants to merge 1 commit into
lightning:masterfrom
dimitris-t:master
Closed

Fixed typos#306
dimitris-t wants to merge 1 commit into
lightning:masterfrom
dimitris-t:master

Conversation

@dimitris-t
Copy link
Copy Markdown

Fixed various typos in multiple documents.

Fixed various typos in multiple documents.
@rustyrussell
Copy link
Copy Markdown
Collaborator

OK, so massive copywriting happening at the moment; I'll apply this once that's settled down if they're not already all fixed. Thanks!

Copy link
Copy Markdown
Collaborator

@toadlyBroodle toadlyBroodle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, all these changes have already been made and/or the relevant sentences changed altogether in the latest copy of BOLT 4.

@shannona
Copy link
Copy Markdown
Collaborator

shannona commented Dec 8, 2017

I added the fix for BOLT-8 to #311. No one has BOLT-11 right now, so either that one bit can be incorporated, or we can leave this open until I get to BOLT-11, hopefully next week.

rustyrussell pushed a commit that referenced this pull request Dec 10, 2017
Added fix for duplicate word courtesy of @dimitris-t in #306.
@shannona
Copy link
Copy Markdown
Collaborator

I'm now working on BOLT-11 and have verified that this change is in; that should be all the referenced BOLTs, so this can be closed.

@shannona shannona mentioned this pull request Dec 16, 2017
@shannona
Copy link
Copy Markdown
Collaborator

I believe these have now been all accounted for, and this PR can be closed.

rustyrussell pushed a commit that referenced this pull request Dec 20, 2017
More line editing and clarifications, also includes typo-fix from @dimitris-t in #306 and table of contents from @bcongdon in #310.
@dimitris-t
Copy link
Copy Markdown
Author

Closing this PR as its changes went into other commits.

@dimitris-t dimitris-t closed this Dec 27, 2017
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.

4 participants