-
Notifications
You must be signed in to change notification settings - Fork 463
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
Address 1.82 clippy lints #8944
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.
Please don't make any changes to proxy until I had a chance to adjust the lints for 1.82. Also, use<T>
is still experimental in 1.81.
oh indeed it is. going to remove the proxy changes. |
e7484d7
to
63c6470
Compare
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!
If you happen to make another clean-up PR there's one more thing I noticed: in rust-toolchain.toml it should just be: components = ["llvm-tools"]
. The -preview suffix was dropped many releases ago.
3836 tests run: 3722 passed, 0 failed, 114 skipped (full report)Flaky tests (3)Postgres 16
Postgres 14
Code coverage* (full report)
* collected from Rust tests only The comment gets automatically updated with the latest test results
2453312 at 2024-09-06T16:43:04.101Z :recycle: |
Oh indeed, wow: rust-lang/rustup#3578 |
Addresses the clippy lints of the beta 1.82 toolchain.
The
too_long_first_doc_paragraph
lint complained a lot and was addressed separately: #8941