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

Update cardano-cli banner #4816

Merged
merged 1 commit into from
Feb 28, 2023
Merged

Update cardano-cli banner #4816

merged 1 commit into from
Feb 28, 2023

Conversation

abailly-iohk
Copy link
Contributor

The previous blurb was pretty outdated and not reflecting what cardano-cli really is about, this is an attempt at being more generic.

@@ -26,9 +26,9 @@ opts =
Opt.info (parseClientCommand <**> Opt.helper)
( Opt.fullDesc
<> Opt.header
"cardano-cli - utility to support a variety of key\
\ operations (genesis generation, migration,\
\ pretty-printing..) for different system generations."
Copy link
Contributor

Choose a reason for hiding this comment

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

While you're here, can you remove multiline literals from these lines and use semigroup concatenation of strings instead?

Copy link
Contributor

Choose a reason for hiding this comment

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

See #4585

@Jimbo4350
Copy link
Contributor

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 28, 2023

Merge conflict.

The previous blurb was pretty outdated and not reflecting what
cardano-cli really is about, this is an attempt at being more
generic.
@Jimbo4350 Jimbo4350 force-pushed the abailly-iohk/update-cli-banner branch from d19d49d to 2473587 Compare February 28, 2023 16:18
@Jimbo4350
Copy link
Contributor

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 28, 2023

Build succeeded:

@iohk-bors iohk-bors bot merged commit 101e247 into master Feb 28, 2023
@iohk-bors iohk-bors bot deleted the abailly-iohk/update-cli-banner branch February 28, 2023 18:21
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.

3 participants