-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Add regression test for issue #76042 #77151
Conversation
(FWIW, I'd appreciate if you could create such branches in forks, not the main repo... it is somewhat confusing to have these things how up in my |
Oops, sorry, I didn’t even notice. I just used GitHub UI to create the commit and forgot I had write access :) |
r=me once the test is actually updated in the manner described in #77151 (comment) |
fe604c5
to
5c2256c
Compare
@pnkfelix Updated the test |
5c2256c
to
e8bda8d
Compare
@bors r+ rollup |
📌 Commit e8bda8d has been approved by |
…laumeGomez Rollup of 6 pull requests Successful merges: - rust-lang#77151 (Add regression test for issue rust-lang#76042) - rust-lang#77996 (Doc change: Remove mention of `fnv` in HashMap) - rust-lang#78463 (Add type to `ConstKind::Placeholder`) - rust-lang#78984 (Rustdoc check option) - rust-lang#78985 (add dropck test for const params) - rust-lang#78996 (add explicit test for const param promotion) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Originally posted in #76042 (comment).
r? @pnkfelix