Skip to content

Added inspect, used anyhow for errors#7

Merged
zslayton merged 1 commit intomasterfrom
inspect-anyhow
Jan 21, 2021
Merged

Added inspect, used anyhow for errors#7
zslayton merged 1 commit intomasterfrom
inspect-anyhow

Conversation

@zslayton
Copy link
Copy Markdown
Contributor

@zslayton zslayton commented Jan 21, 2021

Description of changes:

This PR adds a beta command called inspect that writes a given binary Ion stream to the console alongside its equivalent
text Ion. It prints the offset and length of each value, as well as any symbol IDs used to encode that value's components.

Help

image

Example

image

Other changes:

  • All commands return an anyhow::Result to simplify bubbling errors up to the user level.
  • Created aliases (CommandRunner, CommandConfig) for cumbersome types related to clap configuration.
  • beta module is now a directory.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

* Created a beta command called `inspect` that writes a given
  binary Ion stream to the console alongside its equivalent
  text Ion. It prints the offset and length of each value, as
  well as any symbol IDs used to encode that value's components.
* All commands return an anyhow::Result to simplify bubbling
  errors up to the user level.
* Created aliases for cumbersome types related to clap
  configuration.
* `beta` module is now a directory.
@zslayton zslayton requested a review from cheqianh January 21, 2021 19:30
Copy link
Copy Markdown
Contributor

@cheqianh cheqianh left a comment

Choose a reason for hiding this comment

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

Played around, that's so cool 😎

@zslayton zslayton merged commit 7b84383 into master Jan 21, 2021
@zslayton zslayton deleted the inspect-anyhow branch January 21, 2021 20:46
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.

2 participants