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

Only style the level label #52

Closed
wants to merge 0 commits into from
Closed

Only style the level label #52

wants to merge 0 commits into from

Conversation

delehef
Copy link
Contributor

@delehef delehef commented Nov 11, 2022

As mentioned in #51, this PR proposes a new coloring scheme:

  • only level/timestamps labels are colorized;
  • to improve ‶Eyeballs Mk I grepping″, those are also bold;
  • a few colors were changed to adhere to more common color scales.

Feel free to merge all or pieces of this PR!

If that fits in your vision of this project, there are some other options I would like to introduce in future PRs:

  • user-customizable labels;
  • user-customizable colors;
  • optional constant-width labels (so that e.g. INFO and DEBUG messages become aligned).

@delehef
Copy link
Contributor Author

delehef commented Nov 19, 2022

Bump

@oknozor oknozor mentioned this pull request Jan 18, 2023
Open
cardoe
cardoe previously approved these changes Jan 31, 2024
View reviewed changes
@cardoe
Copy link
Owner

cardoe commented Jan 31, 2024

Any idea why this is failing?

@cardoe cardoe closed this Jan 31, 2024
@cardoe
Copy link
Owner

cardoe commented Jan 31, 2024

bah I screwed up

@cardoe cardoe mentioned this pull request Jan 31, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cardoe cardoe cardoe left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@delehef @cardoe