-
Notifications
You must be signed in to change notification settings - Fork 7
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
fix: correct links and mentions to the v0.32.0 bitcoin #31
fix: correct links and mentions to the v0.32.0 bitcoin #31
Conversation
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.
utACK 95f1401
Didn't check that all the new docs.rs links work. Not sure how best to do that.
That's the purpose of the Give me until the end of the weekend. |
Done! I don't have the permissions necessary. |
Hmm, I don't see a way to do it for branches outside of the repo. I think I should just ACK this PR and merge it and then we'll run it on master and see if we need to fix anything. |
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.
utACK 586a71c
BTW on master it looks like we have two(?) links to https://docs.rs/secp256k1/0.27.0/secp256k1/struct.Keypair.html which 404s. |
Oh sorry man, I ran it before and didn't comment here. |
I can't find those links to 0.27.0 locally? |
Yes, but this is fixed in this PR, you've ran the I've ran the file checking link manually and here's the results (abbreviated with rust-bitcoin.github.io on storopoli/update-links-v0.32 [?] took 6s
❯ docker run -v ${PWD}:/tmp:ro --rm -i ghcr.io/tcort/markdown-link-check:stable /tmp/cookbook/src/tx_segwit-v0.md
(node:1) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
FILE: /tmp/cookbook/src/tx_segwit-v0.md
<ABBR>
50 links checked.
rust-bitcoin.github.io on storopoli/update-links-v0.32 [?] took 15s
❯ docker run -v ${PWD}:/tmp:ro --rm -i ghcr.io/tcort/markdown-link-check:stable /tmp/cookbook/src/tx_taproot.md
(node:1) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
FILE: /tmp/cookbook/src/tx_taproot.md
<ABBR>
58 links checked. |
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.
ACK 586a71c
Thanks for fixing my mistakes! |
I'm not sure how we remember to not do this same mistake next time, currently either the dev or the reviewer has to remember. If I'm the dev again next time there is a reasonable chance I'll forget ... |
I'll try to help the burden and enforce these in the future as well |
In the
Cargo.toml
of the tests we are already using0.32.0
.Hence, the doc links and mentions from
0.31.1
should be updated.PS: I've added a minor inconsistency fix in 95f1401