Skip to content
This repository was archived by the owner on Jan 28, 2026. It is now read-only.

handles err on writing an event#32

Merged
brunocalza merged 1 commit intomainfrom
bcalza/handleerr
Jan 11, 2024
Merged

handles err on writing an event#32
brunocalza merged 1 commit intomainfrom
bcalza/handleerr

Conversation

@brunocalza
Copy link
Contributor

When writing an event, the CLI was capturing errors in the HTTP call. That gave the false feeling that the event was uploaded with success, but it was not.

This PR also removes the SIZE column, when listing events.

Signed-off-by: Bruno Calza <brunoangelicalza@gmail.com>
@brunocalza brunocalza requested a review from dtbuchholz January 11, 2024 14:06
@brunocalza brunocalza marked this pull request as ready for review January 11, 2024 14:08
Copy link
Contributor

@dtbuchholz dtbuchholz left a comment

Choose a reason for hiding this comment

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

lgtm!

@brunocalza brunocalza merged commit 4043f3b into main Jan 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants