-
Notifications
You must be signed in to change notification settings - Fork 720
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
cardano-testnet: QoL changes #6008
Conversation
5a845ea
to
b87724e
Compare
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.
One minor comment.
@@ -65,7 +65,7 @@ pCardanoTestnetCliOptions envCli = CardanoTestnetOptions | |||
pNumSpoNodes :: Parser [TestnetNodeOptions] | |||
pNumSpoNodes = | |||
OA.option | |||
((`L.replicate` SpoTestnetNodeOptions Nothing []) <$> auto) | |||
((`L.replicate` TestnetNodeOptions Nothing []) <$> auto) |
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.
We should keep the differentiation between SPOs and relays.
The point of this PR was actually implemented in #6007, so this one becomes mostly empty. |
e0b10a1
to
0f46d78
Compare
@@ -65,6 +65,8 @@ pCardanoTestnetCliOptions envCli = CardanoTestnetOptions | |||
|
|||
pNumSpoNodes :: Parser [TestnetNodeOptions] | |||
pNumSpoNodes = | |||
-- We don't support passing custom node configurations files on the CLI. |
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.
👍
@@ -144,7 +144,7 @@ getDefaultShelleyGenesis asbe maxSupply opts = do | |||
-- > │ └── README.md | |||
-- > ├── drep-keys | |||
-- > │ ├── drep{1,2,3} | |||
-- > │ │ └── drep.{skey,drep.vkey} | |||
-- > │ │ └── drep.{skey,vkey} |
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.
How did you notice to change this?
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.
It's @carbolymer who did this change, I think after seeing my (late) review on #6007 😉
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.
2c8a630
to
7478ef1
Compare
7478ef1
to
37fcfaa
Compare
37fcfaa
to
31cd564
Compare
[edit] QoL improvements, because most of this PR's content has been concurrently implemented in #6007