doc: fix GitHub links #22115

pull fmarier wants to merge 1 commits into bitcoin:master from fmarier:update-github-links changing 5 files +6 −6

fmarier commented at 2:31 AM on June 1, 2021: none

All of these links now redirect to a new GitHub user.

While the links still work, the fact that they rely on a 301 redirection means that it's possible that some of these repos could be hijacked by someone in the future.
fanquake added the label Docs on Jun 1, 2021

doc: Fix GitHub links

All of these links now redirect to a new GitHub user.

0ee2843fde

fmarier commented at 4:46 AM on June 1, 2021: none

I also took the liberty to fix a HTTP link to GitHub. Happy to remove that if you'd rather not mix the two fixes together.

in doc/release-notes/release-notes-0.8.6.md:66 in 0ee2843fde

  62 | @@ -63,4 +63,4 @@ Warning
  63 |    Hence it is recommended to use a 64-bit executable if possible.
  64 |    A 64-bit executable for Windows is planned for 0.9.
  65 |  
  66 | -Note: Gavin Andresen's GPG signing key for SHA256SUMS.asc has been changed from  key id 1FC730C1 to sub key 7BF6E212 (see https://github.com/bitcoin/bitcoin.org/pull/279).
  67 | +Note: Gavin Andresen's GPG signing key for SHA256SUMS.asc has been changed from  key id 1FC730C1 to sub key 7BF6E212 (see https://github.com/bitcoin-dot-org/Bitcoin.org/pull/279).

fanquake commented at 5:00 AM on June 1, 2021:

Please don't modify historical release notes.

in src/univalue/configure.ac:18 in 0ee2843fde
```
  14 | @@ -15,7 +15,7 @@ m4_define([libunivalue_version], [libunivalue_major_version().libunivalue_minor_
  15 |  
  16 |  
  17 |  AC_INIT([univalue], [1.0.3],
  18 | -        [http://github.com/jgarzik/univalue/])
  19 | +        [https://github.com/jgarzik/univalue/])
```
fanquake commented at 5:01 AM on June 1, 2021:

You can't modify our subtrees here, if you'd like you change this link you'll need to do it upstream: https://github.com/bitcoin-core/univalue.

fmarier commented at 5:11 AM on June 1, 2021:

Ah, I had not realized these were subtrees. Thanks. I will do it in the upstream repos.

fmarier commented at 5:25 AM on June 1, 2021:

https://github.com/jgarzik/univalue/pull/81

in src/bench/nanobench.h:1892 in 0ee2843fde

1888 | @@ -1889,7 +1889,7 @@ void gatherStabilityInformation(std::vector<std::string>& warnings, std::vector<
1889 |          recommendations.emplace_back("Make sure you compile for Release");
1890 |      }
1891 |      if (recommendPyPerf) {
1892 | -        recommendations.emplace_back("Use 'pyperf system tune' before benchmarking. See https://github.com/vstinner/pyperf");
1893 | +        recommendations.emplace_back("Use 'pyperf system tune' before benchmarking. See https://github.com/psf/pyperf");

MarcoFalke commented at 5:21 AM on June 1, 2021:

#22082

in src/crc32c/.ycm_extra_conf.py:7 in 0ee2843fde

   3 | @@ -4,10 +4,10 @@
   4 |  """YouCompleteMe configuration that interprets a .clang_complete file.
   5 |  
   6 |  This module implementes the YouCompleteMe configuration API documented at:
   7 | -https://github.com/Valloric/ycmd#ycm_extra_confpy-specification
   8 | +https://github.com/ycm-core/ycmd#ycm_extra_confpy-specification

fmarier commented at 5:31 AM on June 1, 2021:

https://github.com/google/crc32c/pull/50

fmarier commented at 5:32 AM on June 1, 2021: none

I apologize for the unnecessary pull request. All of this needs to be done in upstream repos.
fmarier closed this on Jun 1, 2021
fmarier deleted the branch on Jun 1, 2021
DrahtBot locked this on Aug 16, 2022

Contributors

Labels