-
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
Regression tests for Rustup breakage tracking issue #3913 #3936
Conversation
Failures: https://travis-ci.com/rust-lang/rust-clippy/jobs/191751335#L1226-L1538 Closing this PR to keep the queue clean. This can be reopened to rerun Travis. |
lets see if #4082 has fixed the identity conversion bug |
Yes identity conversion is fixed!
|
fd13cb7
to
2420eab
Compare
I think 19e852c is also fixed. |
Nice everything is fixed. The tests for this already exist in rust-clippy/tests/ui/into_iter_on_ref.rs Line 13 in bd39cea
rust-clippy/tests/ui/into_iter_on_ref.stderr Lines 1 to 6 in bd39cea
and rust-clippy/tests/ui/for_loop.rs Line 107 in bd39cea
rust-clippy/tests/ui/for_loop.stderr Lines 65 to 71 in bd39cea
|
cc #3913
Tests for the tracking issue. This commits can be cherry-picked for fixing these issues.