-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
Add a regression test for issue 129865 #149421
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
Conversation
|
Thanks! |
Rollup of 5 pull requests Successful merges: - #147362 (Avoid suggesting constrain the associated type with unknown type) - #149395 (float::minimum/maximum: say which exact IEEE operation this corresponds to) - #149396 (Remove outdated comment) - #149421 (Add a regression test for issue 129865) - #149424 (Update books) r? `@ghost` `@rustbot` modify labels: rollup
| //@ compile-flags: -Znext-solver | ||
| //@ check-pass | ||
| //@ edition:2021 | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would have been nice to add a "// Regression test for #129865. <one line explanation of what happened here" to test test.
That makes it a lot easier to figure out what it's supposed to be testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I'll add it when opening next PR
Closes #129865
Looks like the previous versions (
< 1.85.0) failed to normalize async block's upvar types containing constants but not anymore