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.

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

Update example image to how it is currently rendered #596

Merged
merged 1 commit into from
Oct 13, 2023
Merged

Conversation

mitar
Copy link
Contributor

@mitar mitar commented Oct 13, 2023

I noticed that the image is not how currently default console writer renders it.

The image was done using Ubuntu 20.04 using Gnome Terminal (I think default color settings).

I also made a JSONL file with log entries to be able to generate this in the future using the prettylog tool.

BTW, prettylog tool does not work on files, see my comment here.

@mitar mitar force-pushed the image branch 2 times, most recently from cc31f30 to 8ac5965 Compare October 13, 2023 11:41
@mitar mitar marked this pull request as draft October 13, 2023 11:44
@mitar mitar marked this pull request as ready for review October 13, 2023 11:49
@rs
Copy link
Owner

rs commented Oct 13, 2023

I added support for file input in 6e8841b

@rs rs merged commit f70fcca into rs:master Oct 13, 2023
5 checks passed
@mitar
Copy link
Contributor Author

mitar commented Oct 13, 2023

Awesome!

@mitar
Copy link
Contributor Author

mitar commented Oct 13, 2023

One question. I noticed this because I wanted to compare how my changed colors compare to the default colors. I think my changed colors are much more readable. I wonder if this is something you would entertain that I make a PR and change also for defaults in zerolog:

Screenshot from 2023-10-13 14-12-06

Changes:

  • Levels do not have bold to not draw focus.
  • Debug level is not yellow but gray, while warning is yellow (it was really confusing to me).
  • Messages are bold (to draw focus) at info level and above.
  • Error message are bold (to draw focus).

This makes debug and trace messages not stand out (they are all gray) while important ones do (especially important because there are generally many more debug ones).

@rs
Copy link
Owner

rs commented Oct 13, 2023

sure go ahead

madkins23 pushed a commit to madkins23/zerolog that referenced this pull request Mar 2, 2024
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