-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Failure is not an option #62113
Failure is not an option #62113
Conversation
Are you sure you know what you just got into? (It's possible that some tests are there by accident though.) |
This comment has been minimized.
This comment has been minimized.
c260d4f
to
3282050
Compare
Well... now I do... 😂 I just thought they were in
Hmm... so maybe let's move them more incrementally and see which parts stick? |
Yeah, it would be nice to try and see what CI says. |
cc @davidtwco |
I never did revisit these tests and attempt anything to get them working on the |
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 |
I think usually it is a
Did the errors have anything in common? e.g. were they all from a particular part of the compiler? Is there something we can focus on to get the most bang for the buck? |
I hear it can also be an
I'll save this branch into another for now and remove some bits from the PR and see if we can merge that. Once we have determined the set of tests that cannot be moved we can consider fixes for those. |
Unfortunately, I can’t remember. I’m afraid I didn’t keep notes as to what the failures were at the time, but you could probably look at old Travis logs from the original PR, if they are still around. |
Extracted a subset of this PR into #62177. Marking as blocking progress on that. |
…s, r=petrochenkov Move some tests in src/test/compile-fail -> src/test/ui This is a subset of rust-lang#62113. Hopefully everything here sticks, but if it doesn't I'll remove tests until it does. r? @petrochenkov
☔ The latest upstream changes (presumably #62355) made this pull request unmergeable. Please resolve the merge conflicts. |
#62177 was closed, so closing this as well. |
Remove the
src/test/compile-fail
folder and move all of its tests into appropriate places insrc/test/ui
. Some tests insrc/test/ui
are also moved as a result for better categorization.r? @petrochenkov