-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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 #[test]
to functions in test modules
#99036
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
r? @Dylan-DPC |
r? @rust-lang/compiler |
the correct invocation is: r? rust-lang/compiler but i will take a look at this tomorrow if nobody else does. |
and then high-five actually does the fake command instead of the real one. d'oh. |
Haha, sorry, I self-assigned this but forgot to review it. Looks fine to me. @bors r+ rollup |
Rollup of 7 pull requests Successful merges: - rust-lang#94927 (Stabilize `let_chains` in Rust 1.64) - rust-lang#97915 (Implement `fmt::Write` for `OsString`) - rust-lang#99036 (Add `#[test]` to functions in test modules) - rust-lang#99088 (Document and stabilize process_set_process_group) - rust-lang#99302 (add tracking issue to generic member access APIs) - rust-lang#99306 (Stabilize `future_poll_fn`) - rust-lang#99354 (Add regression test for rust-lang#95829) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
I implemented a suggestion in #91770, but the ui test I created was inadequate and I have fixed that.