-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
regression for issue 90847 #93389
regression for issue 90847 #93389
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Mark-Simulacrum (or someone else) soon. Please see the contribution instructions for more information. |
This test seems like it may be in an odd directory? Could you move it to src/test/ui/const-generics/generic_const_exprs? |
18bf17a
to
9118393
Compare
Ping from triage: FYI: when a PR is ready for review, send a message containing @rustbot ready |
Ah thanks for letting me know, didn't know about that 😁 |
@bors r+ rollup |
📌 Commit 9118393 has been approved by |
… r=Mark-Simulacrum regression for issue 90847 Adds a regression test for issue rust-lang#90847
…askrgr Rollup of 5 pull requests Successful merges: - rust-lang#89793 (Add `slice::{from_ptr_range, from_mut_ptr_range} `) - rust-lang#92642 (Update search location from a relative path to absolute) - rust-lang#93389 (regression for issue 90847) - rust-lang#93413 (Fix broken link from rustdoc docs to ayu theme) - rust-lang#94365 (Fix MinGW target detection in raw-dylib) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Adds a regression test for issue #90847