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

Prefer Shelley CLI commands over Byron ones where they clash #2145

Merged
merged 1 commit into from
Nov 30, 2020

Conversation

dcoutts
Copy link
Contributor

@dcoutts dcoutts commented Nov 30, 2020

We have done a fair bit of shuffling of CLI commands over time, and have
kept the old commands for backwards compatibility.

There was a name clash over the "genesis" top level command: between the
Shelley command and the backwards compat Byron command.

Reorder the command lists so we prefer the new Shelley commands, and
generally the new ones over the old ones.

Copy link
Contributor

@mkoura mkoura left a comment

Choose a reason for hiding this comment

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

👍

We have done a fair bit of shuffling of CLI commands over time, and have
kept the old commands for backwards compatibility.

There was a name clash over the "genesis" top level command: between the
Shelley command and the backwards compat Byron command.

Reorder the command lists so we prefer the new Shelley commands, and
generally the new ones over the old ones.
@dcoutts dcoutts force-pushed the dcoutts/cli-fix-command-precedence branch from 5cd1481 to 8c29016 Compare November 30, 2020 11:40
@dcoutts
Copy link
Contributor Author

dcoutts commented Nov 30, 2020

bors merge

iohk-bors bot added a commit that referenced this pull request Nov 30, 2020
2145: Prefer Shelley CLI commands over Byron ones where they clash r=dcoutts a=dcoutts

We have done a fair bit of shuffling of CLI commands over time, and have
kept the old commands for backwards compatibility.

There was a name clash over the "genesis" top level command: between the
Shelley command and the backwards compat Byron command.

Reorder the command lists so we prefer the new Shelley commands, and
generally the new ones over the old ones.

Co-authored-by: Duncan Coutts <duncan@well-typed.com>
@dcoutts
Copy link
Contributor Author

dcoutts commented Nov 30, 2020

bors merge

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Nov 30, 2020

@iohk-bors iohk-bors bot merged commit 0dac653 into master Nov 30, 2020
@iohk-bors iohk-bors bot deleted the dcoutts/cli-fix-command-precedence branch November 30, 2020 16:04
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