-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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 clippy #64871
Update clippy #64871
Conversation
|
@bors r+ p=1 |
📌 Commit 7f8248726debad62d02cabdec5557b3ff595dddd has been approved by |
⌛ Testing commit 7f8248726debad62d02cabdec5557b3ff595dddd with merge fc6384337fd0707009950b7ab0bd14e8dad6fa18... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-azure |
Hmm, it seems needed more fix, unrelated clippy? |
#64900 overrides this so I'm going to close. |
Ah, it doesn't resolve the problem so I re-open this. |
@oli-obk This output is from the log:
An ICE is unrelated to clippy? |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
7f82487
to
0e98559
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Hmm, oddly, some stderr have gone. |
Seems to be caused by Clippy. Tests don't work because compiler crashes. |
These seem to be new failures caused in the tests since the last clippy update, i.e something changed in the compiler in the time it took for us to do the clippyup. |
It was broken by something in #64902 |
It seems some loop lints are broken. |
Figured it out. It was #64825 |
will open a new PR |
Closes #64867
CC rust-lang/rust-clippy#4594
r? @oli-obk @Manishearth