Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

docs: update README.md to clarify install instructions #33

Merged
merged 5 commits into from
May 11, 2020
Merged

docs: update README.md to clarify install instructions #33

merged 5 commits into from
May 11, 2020

Conversation

cloudwheels
Copy link
Contributor

Docs only updated to address issues #10 and #32 (close on merge)

Oops. Additonal -> Additional (in new updates)
Consistent capitals for 'Docker Compose', and lower case for 'masternode' and 'full node'
@cloudwheels
Copy link
Contributor Author

Actually I wasn't aware of the work on #30 so the updates related to MN registration may not be appropriate.

README.md Outdated
> Distribution package for Dash Masternode installation
> Distribution package for Dash masternode installation

> Note: This Docker Compose project creates a full node. Additional steps are required to register a masternode, as described in the [setup instructions](https://docs.dash.org/en/stable/masternodes/setup.html#register-your-masternode). It is not currently necessary to complete these registration steps to test development with Dash Platform services or participate as a full node on Evonet.
Copy link
Member

Choose a reason for hiding this comment

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

I would reword this part "test development" -> "local development" since we have "local" preset

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@shumkov see 89909c6

(and minor change b61b110)

Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

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

Great! Thank you!

@shumkov shumkov merged commit 6840d51 into dashevo:master May 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants