-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Clippy subtree update #132746
Clippy subtree update #132746
Conversation
pulldown-cmark: don't pull getopts dep Don't pull getopts, as it unused. Noticed in rust-lang#131892 changelog: none
Allow to go through clippy lints page without javascript Fixes rust-lang#13536. This is the follow-up of rust-lang/rust-clippy#13269. This PR makes it possible to expand/collapse lints (individually) without JS. To achieve this result, there are two ways: 1. Use `details` and `summary` tags. Problem with this approach is that the web browser search may open the `details` tags automatically if content matching it is inside. From a previous discussion with `@Alexendoo,` it seems to not be a desired behaviour. 2. Use a little trick where you use a `label` and a checkbox where the checkbox is in fact hidden. Then it's just a matter of CSS. r? `@Alexendoo` changelog: Allow to go through clippy lints page without JS
Add internal lint to check for slow symbol comparisons See the conversation on [Zulip](https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/Checking.20a.20Symbol.20is.20equal.20to.20a.20string.20literal). changelog: none
…blyxyas Stop linting manual_bits in any macro invoke Closes rust-lang#13563. changelog: [`manual_bits`] No longer lints in macro-generated code
Before this change, adding a lint was a difficult matter because it always had some overhead involved. This was because all lints would run, no matter their default level, or if the user had #![allow]ed them. This PR changes that
Need
|
Looks like we never build Clippy in release mode in the Clippy repo. Should be good now. |
This comment has been minimized.
This comment has been minimized.
6c87427
to
0aafd65
Compare
@bors r=Manishearth |
☀️ Test successful - checks-actions |
Finished benchmarking commit (59cec72): comparison URL. Overall result: ❌ regressions - please read the text belowOur benchmarks found a performance regression caused by this PR. Next Steps:
@rustbot label: +perf-regression Instruction countThis is the most reliable metric that we have; it was used to determine the overall result at the top of this comment. However, even this metric can sometimes exhibit noise.
Max RSS (memory usage)Results (secondary -1.4%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResults (secondary 2.1%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 780.103s -> 779.98s (-0.02%) |
This doesn't change the compiler, and @rustbot label: +perf-regression-triaged |
@jdonszelmann this includes the Clippy attribute changes, you wanted. |
Amazing ,thanks so much! |
…nishearth Clippy subtree update r? `@Manishearth`
r? @Manishearth