Skip to content

Enable verbose option on all commands#4

Merged
zmoog merged 2 commits intomainfrom
zmoog/verbose
Apr 12, 2023
Merged

Enable verbose option on all commands#4
zmoog merged 2 commits intomainfrom
zmoog/verbose

Conversation

@zmoog
Copy link
Copy Markdown
Owner

@zmoog zmoog commented Apr 12, 2023

Motivation

Defaults to printing the message payload only, so it is easier to combine output with other tools, like jq:

$ eh eventdata receive | jq '.records[].tenantId' 
"1de2b364-21e1-4866-bb46-7804f17c417d"
"1de2b364-21e1-4866-bb46-7804f17c417d"
"1de2b364-21e1-4866-bb46-7804f17c417d"

But you can still turn on verbose and get more information about what's going on:

$ eh --verbose eventdata receive
Receiving events from mbranca
Received event from partition 0: {"records": [{....

Change description

Add a couple of ifs.

Reviewer checklist

  • PR address a single concern.
  • PR title and description are appropriately filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.
  • Docs are updated (at least the README.md, if needed).
  • History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well-formatted.

Defaults to printing the message payload only, so it is easier to
combine to output with other tools, like `jq`.
@zmoog zmoog self-assigned this Apr 12, 2023
@zmoog zmoog added the enhancement New feature or request label Apr 12, 2023
@zmoog zmoog marked this pull request as ready for review April 12, 2023 14:42
@zmoog zmoog merged commit ed6a2cb into main Apr 12, 2023
@zmoog zmoog deleted the zmoog/verbose branch April 12, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant