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

Minor fixes to BIP-0009 #1130

Merged
merged 2 commits into from
Jun 12, 2021
Merged

Minor fixes to BIP-0009 #1130

merged 2 commits into from
Jun 12, 2021

Conversation

tcharding
Copy link
Contributor

Do two minor fixes to BIP-0009

  • Add a couple of extra hyperlinks to other bip docs to assist the reader
  • Fix typo, remove unneeded 'in'

When mentioning a bip we use a hyperlink to the bip file to assist
readers. There are two mentions of bips in BIP-0009 that do not use
links, probably because links where provided above in the text, however,
its a bit easier on the reader if we provide links again to save
scrolling through the document.

Use hyperlinks when mentioning BIP 34 and 141 in isolation.
Phrase 'based on in BIP 43' should probably read 'based on BIP 34'.
@luke-jr
Copy link
Member

luke-jr commented Jun 12, 2021

@sipa
Copy link
Member

sipa commented Jun 12, 2021

ACK f59b209

@luke-jr luke-jr merged commit 4493524 into bitcoin:master Jun 12, 2021
@tcharding tcharding deleted the add-bip-links branch June 13, 2021 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants