-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Extend unnecessary_lazy_eval to cover bool::then
-> bool::then_some
#9099
Extend unnecessary_lazy_eval to cover bool::then
-> bool::then_some
#9099
Conversation
r? @llogiq (rust-highfive has picked a reviewer for you, use r? to override) |
This is WIP because I haven't managed to get the change to actually fire on the test case; I would appreciate advice on what I'm missing. |
This should probably have a MSRV check for 1.62. |
12eb1d3
to
9059bb2
Compare
|
9059bb2
to
b7230d4
Compare
@flip1995 Done. |
@bors r+ Thanks! |
📌 Commit b7230d4 has been approved by |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
fixes #9097
changelog: Extend
unnecessary_lazy_eval
to convertbool::then
tobool::then_some