-
Notifications
You must be signed in to change notification settings - Fork 400
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
chore: clippy #1106
chore: clippy #1106
Conversation
rust/src/writer/mod.rs
Outdated
#[from] | ||
stats: serde_json::Error, |
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 the only manual change. Clippy was complaining that the error variant was much larger then all the others. I felt forwarding the error rather then the actual data that caused the error is a reasonable change.
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.
That seems fair. The other option is to Box
the Stats, but I think the serde_json
error should be sufficient.
Could we update the doc comment above, though?
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.
Of course! Going forward we way consider lifting the docs requirement for error variants. I feel the error description text and variant name are usually expressive enough.
# Description The latest rust release comes with new more opinionated clippy :). This PR fixes the new clippy errors and and runs `cargo clippy --fix` on all our crates. # Related Issue(s) <!--- For example: - closes delta-io#106 ---> # Documentation <!--- Share links to useful documentation --->
Description
The latest rust release comes with new more opinionated clippy :). This PR fixes the new clippy errors and and runs
cargo clippy --fix
on all our crates.Related Issue(s)
Documentation