Skip to content

feature: add customization of format levels#21

Merged
davidbarsky merged 2 commits intodavidbarsky:mainfrom
markbt:format_level_chars
Jul 10, 2025
Merged

feature: add customization of format levels#21
davidbarsky merged 2 commits intodavidbarsky:mainfrom
markbt:format_level_chars

Conversation

@markbt
Copy link
Contributor

@markbt markbt commented Jul 3, 2025

Classic glog doesn't use D or T for debug or trace, it uses V for verbose. Let the caller customize the letters so that they can use this instead.

Classic glog doesn't use D or T for debug or trace, it uses V for
verbose.  Let the caller customize the letters so that they can use this
instead.
@davidbarsky
Copy link
Owner

Looks good to me! mind fixing the clippy lint?

@markbt
Copy link
Contributor Author

markbt commented Jul 7, 2025

The clippy lints are pre-existing (they are all complaining that "variables can be used directly in the format! string"). I can fix if you want but it will bump MSRV 1.58 (or we can allow(clippy::uninlined_format_args, or set the rust version in Cargo.toml to suppress the lints).

Change all format strings to inline the format args where possible.  This requires Rust 1.58. so set `rust-version` accordingly.
@markbt
Copy link
Contributor Author

markbt commented Jul 10, 2025

Updated the PR to fix the clippy lints and set the MSRV.

@davidbarsky davidbarsky merged commit 8638269 into davidbarsky:main Jul 10, 2025
5 checks passed
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