-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[Merged by Bors] - add documentation on LogPlugin and more log usage #1973
Conversation
} else { | ||
println!("sphere hasn't loaded yet"); | ||
info!("sphere hasn't loaded yet"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is nitpicking but there is some inconsistency in examples now whether capitalization is used at the start for the log messages.
E.g.
"sphere hasn't loaded yet"
vs.
""Custom asset loaded: {:?}"
in custom_asset.rs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think starting with a capital is the right call. Feel free to follow up with another pr that makes this consistent if you feel inclined :)
bors r+ |
Fixes #1895 Changed most `println` to `info` in examples, some to `warn` when it was useful to differentiate from other more noisy logs. Added doc on `LogPlugin`, how to configure it, and why (and how) you may need to disable it
Pull request successfully merged into main. Build succeeded: |
Fixes bevyengine#1895 Changed most `println` to `info` in examples, some to `warn` when it was useful to differentiate from other more noisy logs. Added doc on `LogPlugin`, how to configure it, and why (and how) you may need to disable it
Fixes #1895
Changed most
println
toinfo
in examples, some towarn
when it was useful to differentiate from other more noisy logs.Added doc on
LogPlugin
, how to configure it, and why (and how) you may need to disable it