-
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-test: small improvements #5962
Conversation
1753af9
to
0d6082e
Compare
80fb3ee
to
81c84f7
Compare
81c84f7
to
a245cd5
Compare
Are we using P2P stack in tests? Note that the non-p2p stack will be removed at some point. |
@coot> I believe we don't use P2p in → cat /tmp/nix-shell.3H7AVw/cli-queries-test-23d51bd100cd79ee/configuration.yaml | grep P2P
"EnableP2P": false, |
Let me amend the PR then, to keep the flag then and honor it instead of doing nothing with it 🙂 |
Honoring P2P will affect other work I have stacked on this one, so I'll just keep the "small improvements" bits of this PR and merge. P2P honoring will come later. |
a245cd5
to
47cc7cc
Compare
Note that |
Description
This PR removes initialization of fields in
cardano-testnet-test
that were useless because they were initializing said fields with the same value as the default values. It also does QoL improvements.Checklist