-
Notifications
You must be signed in to change notification settings - Fork 130
Updated to remove unneeded step to start and stop Pantheon #595
Conversation
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.
Removing the start/stop/restart simplifies this quite a bit. The revised procedure steps are clear, just one suggestion in comments.
@@ -21,8 +21,8 @@ To create a private network: | |||
|
|||
1. [Create Folders](#1-create-folders) | |||
1. [Create Genesis File](#2-create-genesis-file) | |||
1. [Start First Node and Get Node Public Key](#3-start-first-node-and-get-node-public-key) | |||
1. [Restart First Node as Bootnode](#4-restart-first-node-as-bootnode) | |||
1. [Get Node Public Key of First Node](#3-start-first-node-and-get-node-public-key) |
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.
Could this step say "Get Public Key of First Node"?
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.
Indeed yes, it doesn't start the node, only exports the key
Also if **3. Start First Node and Get Node Public Key ** is renamed to 3. Get Node Public Key of First Node, the link will change too, so need to be #3-get-node-public-key-of-first-node
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.
Done
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.
Done
@@ -80,34 +80,13 @@ Copy the following genesis definition to a file called `privateNetworkGenesis.js | |||
Do not use the accounts in the genesis file above on mainnet or any public network except for testing. | |||
The private keys are displayed so the accounts are not secure. | |||
|
|||
### 3. Start First Node and Get Node Public Key | |||
### 3. Get Node Public Key of First Node |
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.
(Same comment as above)
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.
Done
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.
Done
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.
Link to be fixed
@@ -21,8 +21,8 @@ To create a private network: | |||
|
|||
1. [Create Folders](#1-create-folders) | |||
1. [Create Genesis File](#2-create-genesis-file) | |||
1. [Start First Node and Get Node Public Key](#3-start-first-node-and-get-node-public-key) | |||
1. [Restart First Node as Bootnode](#4-restart-first-node-as-bootnode) | |||
1. [Get Node Public Key of First Node](#3-start-first-node-and-get-node-public-key) |
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.
Indeed yes, it doesn't start the node, only exports the key
Also if **3. Start First Node and Get Node Public Key ** is renamed to 3. Get Node Public Key of First Node, the link will change too, so need to be #3-get-node-public-key-of-first-node
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.
Lgtm
Fixed Issue(s)
fixes #525