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.

Sign up for GitHub

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

Jump to bottom

Tidy up the CLI tx out datum support #2806

Merged
merged 4 commits into from
Jun 9, 2021
Merged

Tidy up the CLI tx out datum support #2806

merged 4 commits into from
Jun 9, 2021

Commits on Jun 9, 2021

  1. Consolodate most of the CLI parsers onto parsec 

    Rather than a mix of attoparsec and parsec.

attoparsec is still used for Scientific values and JSON.
    @dcoutts
    dcoutts committed Jun 9, 2021
    Configuration menu
    Copy the full SHA
    0dfa8a5 View commit details
    Browse the repository at this point in the history

  2. Tidy up the handling of tx out datums in the CLI 

    Simpler story with parsing, and with the internal handling. Parse it
fully in the CLI parser so we don't need to parse later.

Also properly report the use of output datums when it is not supported,
rather than silently ignoring them.
    @dcoutts
    dcoutts committed Jun 9, 2021
    Configuration menu
    Copy the full SHA
    028f19b View commit details
    Browse the repository at this point in the history

  3. Rename --datum-hash flag to --tx-out-datum-hash 

    For consistency with the --tx-out flag.

This has not been released yet, so there are no backwards compat
concerns.
    @dcoutts
    dcoutts committed Jun 9, 2021
    Configuration menu
    Copy the full SHA
    09006af View commit details
    Browse the repository at this point in the history

  4. Suppress an unwanted hlint warning

    @dcoutts
    dcoutts committed Jun 9, 2021
    Configuration menu
    Copy the full SHA
    04cf579 View commit details
    Browse the repository at this point in the history