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

doc: Rework internal and external links #21210

Merged
merged 1 commit into from
Feb 19, 2021
Merged

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Feb 17, 2021

Some minor changes:

  • Move Bitcoin Core download link to the very top. Reason: The download link has nothing to do with the section that explains Bitcoin. Also, anyone quickly looking for the download link will find it faster.
  • Add a new link to the doc folder. Reason: Apart from the documentation that is shipped with the binary software, the doc folder is the primary location for Bitcoin Core related documentation.
  • Remove dead link to pdf. Reason: The pdf can be trivially found by asking a search engine.
  • Remove reference to "build server". Reason: There is no "build server". The CI system is explained in the next sentence in detail.
  • Remove dead? link to translation mailing list. Reason: The translation process is explained in doc/translation_process.md, no need to explain it in detail in the main readme.

@fanquake fanquake added the Docs label Feb 17, 2021
README.md Show resolved Hide resolved
@DrahtBot
Copy link
Contributor

DrahtBot commented Feb 17, 2021

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@laanwj
Copy link
Member

laanwj commented Feb 18, 2021

ACK 77772a1

Remove dead? link to translation mailing list. Reason: The translation process is explained in doc/translation_process.md, no need to explain it in detail in the main readme.

Good find. I can't even remember anymore when we used that list.

Copy link
Contributor

@RiccardoMasutti RiccardoMasutti left a comment

Choose a reason for hiding this comment

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

ACK 77772a1

@laanwj laanwj merged commit 56fe4bb into bitcoin:master Feb 19, 2021
@maflcko maflcko deleted the 2102-docLinks branch February 19, 2021 10:16
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Feb 19, 2021
@ghost ghost mentioned this pull request Feb 20, 2021
@bitcoin bitcoin locked and limited conversation to collaborators Feb 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants