-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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 two tests for changes in rustc output #7231
Conversation
r? @ehuss (rust_highfive has picked a reviewer for you, use r? to override) |
The easiest thing here is probably to relax the assertion by changing |
c8f2c5b
to
6959316
Compare
Updated with |
@bors: r+ |
📌 Commit 6959316 has been approved by |
Update two tests for changes in rustc output Required for landing rust-lang/rust#63250. (I'm not sure what is the process for landing synchronous changes like this.)
☀️ Test successful - checks-azure |
@petrochenkov Feel free to update the cargo submodule in your PR if you think it will get merged within the next week or so. Or if you'd prefer, I can post an update PR, but you'll have to wait for it to make its way through the bors queue. Just let me know. |
Required for landing rust-lang/rust#63250.
(I'm not sure what is the process for landing synchronous changes like this.)