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
[docs] Fixed outdated link with archive.is #11804
Conversation
The listed link is directing to an empty page, at least content-wise. I found the same page on archive.is and linked to that instead.
@@ -52,4 +52,4 @@ examine `uint256_tests.cpp`. | |||
|
|||
For further reading, I found the following website to be helpful in | |||
explaining how the boost unit test framework works: | |||
[http://www.alittlemadness.com/2009/03/31/c-unit-testing-with-boosttest/](http://www.alittlemadness.com/2009/03/31/c-unit-testing-with-boosttest/). |
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.
Nice catch!
How about pointing to official documentation? Also, this paragraph could be improved to remove the first person.
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.
I don't think he has to fix all the surrounding issues in the file just for fixing a link.
IMO this PR is fine as-is. But if you want to make further updates sure!
ACK |
bf20a7d [docs] Fixed outdated link with archive.is (Tim Shimmin) Tree-SHA512: 7d316aa4c462213578a9a1e71b06a459924ed3bb681a0469cc3719b21a871a6350fff4656696057e362561ee10dc10e933fe1328ef454ab7e133ecf05549ec1c
bf20a7d [docs] Fixed outdated link with archive.is (Tim Shimmin) Tree-SHA512: 7d316aa4c462213578a9a1e71b06a459924ed3bb681a0469cc3719b21a871a6350fff4656696057e362561ee10dc10e933fe1328ef454ab7e133ecf05549ec1c
bf20a7d [docs] Fixed outdated link with archive.is (Tim Shimmin) Tree-SHA512: 7d316aa4c462213578a9a1e71b06a459924ed3bb681a0469cc3719b21a871a6350fff4656696057e362561ee10dc10e933fe1328ef454ab7e133ecf05549ec1c
Summary: bf20a7d [docs] Fixed outdated link with archive.is (Tim Shimmin) --- Backport of Core [[bitcoin/bitcoin#11804 | PR11804]] Test Plan: read it Reviewers: #bitcoin_abc, PiRK, Fabien Reviewed By: #bitcoin_abc, PiRK, Fabien Differential Revision: https://reviews.bitcoinabc.org/D8173
The listed link is directing to an empty page, at least content-wise. I found the same page on archive.is and linked to that instead.