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
Suggested doc fix #340
Suggested doc fix #340
Conversation
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.
Thanks for the fix, I agree that this is clearer! There are a couple tweaks I'd like to make before merging this.
tracing-core/src/field.rs
Outdated
@@ -219,6 +219,8 @@ pub trait Value: crate::sealed::Sealed { | |||
} | |||
|
|||
/// A `Value` which serializes as a string using `fmt::Display`. |
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.
Perhaps we should also remove the "as a string", as well, if this was a source of confusion?
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.
lgtm!
@samscott89 i believe that if you rebase or merge master, the CI build for this should be fixed. |
Suggested documentation fix for confusion in #306