Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

Updated to remove unneeded step to start and stop Pantheon #595

Merged
merged 22 commits into from
Jan 20, 2019
Merged

Updated to remove unneeded step to start and stop Pantheon #595

merged 22 commits into from
Jan 20, 2019

Conversation

MadelineMurray
Copy link
Contributor

@MadelineMurray MadelineMurray commented Jan 17, 2019

Fixed Issue(s)

fixes #525

Copy link
Contributor

@donntrenton donntrenton left a 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)
Copy link
Contributor

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"?

Copy link
Contributor

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor Author

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
Copy link
Contributor

Choose a reason for hiding this comment

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

(Same comment as above)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@NicolasMassart NicolasMassart left a 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)
Copy link
Contributor

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

Copy link
Contributor

@NicolasMassart NicolasMassart left a comment

Choose a reason for hiding this comment

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

Lgtm

@MadelineMurray MadelineMurray merged commit 246d6be into PegaSysEng:master Jan 20, 2019
@MadelineMurray MadelineMurray deleted the genNodeKey branch January 20, 2019 16:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants