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

bip38 typo: specifid -> specified #608

Merged
merged 1 commit into from
Apr 22, 2024
Merged

Conversation

philsmd
Copy link
Contributor

@philsmd philsmd commented Nov 7, 2017

There was a small typo in the bip38 specification. If I'm not totally mistaken the word should be "specified" (not specifid)
Thx

There was a small typo in the bip38 specification. If I'm not totally mistaken the word should be "specified" (not specifid)
Thx
@luke-jr
Copy link
Member

luke-jr commented Nov 7, 2017

@voisine

Copy link
Contributor

@murchandamus murchandamus left a comment

Choose a reason for hiding this comment

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

ACK b84deb2

@jonathancross
Copy link
Contributor

jonathancross commented Apr 6, 2024

Thanks @murchandamus
This is 6.5 years old... anyone plan to merge it? It's just a small typo fix.
@luke-jr or @jamesob?

@murchandamus
Copy link
Contributor

@jonathancross: I was looking around a bit what sort of pull requests were open in the context of the discussions about adding BIP editors and amending the BIP process. It would be great if we could amend the process to provide an avenue for small changes that don’t affect the meaning of BIPs like this to get merged more easily.

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

ACK b84deb2

@jonatack jonatack merged commit b59384d into bitcoin:master Apr 22, 2024
@philsmd philsmd deleted the patch-1 branch May 7, 2024 09:42
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.

6 participants