-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update to handle all the latest cargo warnings #3720
Conversation
src/uu/factor/src/cli.rs
Outdated
let stdin = stdin(); | ||
|
||
for line in stdin.lock().lines() { | ||
for line in stdin().lines() { |
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.
Error here:
use of unstable library feature 'stdin_forwarders'
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 removed this fix - clippy is not 100% happy still, but at least it is much closer to passing now.
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.
Clippy should take the MSRV into account with the suggestions is makes, so I think we're good. We'll start seeing them once we bump the MSRV.
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.
Doesn't build currently
c9da98b
to
77956b8
Compare
77956b8
to
cc08e1c
Compare
This PR should be good to go, the GnuTests failure is actually inaccurate (it is basing the output on PRs, not on main). |
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.
Looks great!
This PR fixes ~100 different clippy lint warnings. Many of these may have been introduced by the clippy 1.64 release.