-
Notifications
You must be signed in to change notification settings - Fork 13.4k
[beta] backport of readd capture disjoint fields gate #85893
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
(and thus exposed underlying issue rust-lang#85561).
Apply suggestions from code review: 1. (removing confusing comment from my test, since the comment reflects the bad undesirable behavior that is being fixed here.) 2. test THIR unsafeck too. Co-authored-by: Léo Lanteri Thauvin <leseulartichaut@gmail.com>
r? @varkor (rust-highfive has picked a reviewer for you, use r? to override) |
|
This comment has been minimized.
This comment has been minimized.
argh that's what I get for trusting that the test would trivially work when backported. will fix. |
remove testing via `-Z thir-unsafeck`, since it is not appropriate for beta branch.
r=me |
@bors r=simulacrum |
📌 Commit f9f5fc8 has been approved by |
@bors rollup=never |
⌛ Testing commit f9f5fc8 with merge 3554ba4b26dea7c8ee1aa3633b709276b077a994... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry |
⌛ Testing commit f9f5fc8 with merge f676e1dc994234fd9204368679ff9f5b5df7944d... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
⌛ Testing commit f9f5fc8 with merge 78192079381e08ace362f0b640641085d19de010... |
💔 Test failed - checks-actions |
The job Click to see the possible cause of the failure (guessed by this bot)
|
CI issue has been fixed, @bors retry |
☀️ Test successful - checks-actions |
Beta backport of "readd capture disjoint fields gate", PR #85564
Fix issue #85435