-
Notifications
You must be signed in to change notification settings - Fork 33
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: fix clippy 1.81 #323
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 :-)
@boxdot kind ping for the requested changes; maybe it wasn't clear from the notification that it was a change request :'-) |
Thank you for reminding me. I got carried away. |
Lint reasons don't seem to be stabilized yet in 1.75, any chance we can put the reason behind a nightly flag? Sounds a bit wrong to bump MSRV for this :'-) (Sailfish is on 1.75 still, and will be for at least half a year I fear) |
I think it is pragmatic to remove the reasons here. They are kind of obvious from the description of the lint. |
Description and position make it clear, indeed. Thank you! |
No description provided.