Skip to content

Harmonize bail! vs panic! in the CLI #5506

Open
@thibault-martinez

Description

@thibault-martinez
          We can definitely `bail`, but I do see some panics in `validator_commands` already. For example

IotaKeyPair::Ed25519(account_key) => IotaKeyPair::Ed25519(account_key.copy()),
_ => panic!(
"Other account key types supported yet, please use Ed25519 keys for now."
),

So whatever you think is more consistent for the API. cc: @thibault-martinez

Originally posted by @kodemartin in #5502 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    dev-toolsIssues related to the Developer Tools Teamsc-platformIssues related to the Smart Contract Platform group.

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions