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

Add markdown file with droplet test instructions #36

Merged
merged 4 commits into from
Dec 7, 2021

Conversation

jbowen93
Copy link
Contributor

@jbowen93 jbowen93 commented Dec 3, 2021

Adds documentation that explains how to create a dev network out of celestia-app nodes using 3 Digital Ocean droplets.

Copy link
Member

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

Excited about this! Please add a brief description in the opening comment :)

@@ -0,0 +1,99 @@
# Attempt to create network on 3 DO Droplets
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Attempt to create network on 3 DO Droplets
# Create a network on 3 DO Droplets

Comment on lines 30 to 32
node_0_ip=188.166.21.209
node_1_ip=188.166.91.191
node_2_ip=174.138.11.26
Copy link
Member

Choose a reason for hiding this comment

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

Add a comment somewhere that this will require the reader to use their IPs instead (otherwise they will just copy&paste).

Copy link
Member

@Bidon15 Bidon15 left a comment

Choose a reason for hiding this comment

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

Looks awesome!
I've create an issue that can be done later based on your PR
#37

@jbowen93 jbowen93 merged commit ca720b8 into master Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants