-
Notifications
You must be signed in to change notification settings - Fork 288
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: Clarify README.md installation guides. #2171
docs: Clarify README.md installation guides. #2171
Conversation
Regarding the badges at the top of the README: Would it be better to link to the dcrd docs themselves? |
That version refers to the module version which is not the same as the overall software version. It is correct and up-to-date. |
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.
The modifications look good. Now, the commits just need to be squashed into a single commit and updated to match the Code Contribution Guidelines. Namely, the Model Git Commit Messages section.
I went ahead and modified the PR title accordingly.
This updates the primary
README.md
.Latest Downloads:
Added clarifying text to inform new users how dcrd is differentiated from other Decred software.
The purpose is to guide people into the easier software bundles at https://decred.org/downloads, and only continue at dcrd if they truly desire.
What is dcrd?
Added clarifying text that dcrd does not include wallet functionality by itself.
Binaries
Again, trying to direct new users to https://decred.org/downloads, but added additional text to guide them into the docs.
Build from Source
Dependencies:
GOPATH
andGOROOT
are handled by default since Go 1.8, but many old Go tutorials direct users to play in dangerous waters. This text should help clarify that users don't need to mess with anything, and only Unix-based users need to mess with their $PATH environment variable.Further, clarified and reorganized instructions for both Windows and Unix-based users.
Running Tests
This was a sub-header under Docker, so I changed it to be its own header.