-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Add new people to the compiletest review rotation #108125
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add new people to the compiletest review rotation #108125
Conversation
(rustbot has picked a reviewer for you, use r? to override) |
@bors r+ rollup |
@@ -630,7 +630,7 @@ style-team = [ | |||
"/src/stage0.json" = ["bootstrap"] | |||
"/tests/ui" = ["compiler"] | |||
"/src/tools/cargo" = ["@ehuss", "@joshtriplett"] | |||
"/src/tools/compiletest" = ["bootstrap"] | |||
"/src/tools/compiletest" = ["bootstrap", "@wesleywiser", "@oli-obk"] |
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.
I was asleep when this PR was put up, but I can also add myself to this rotation since I have reasonable familiarity with compiletest :)
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.
Go ahead, more is better!
…iaskrgr Rollup of 7 pull requests Successful merges: - rust-lang#106347 (More accurate spans for arg removal suggestion) - rust-lang#108057 (Prevent some attributes from being merged with others on reexports) - rust-lang#108090 (`if $c:expr { Some($r:expr) } else { None }` =>> `$c.then(|| $r)`) - rust-lang#108092 (note issue for feature(packed_bundled_libs)) - rust-lang#108099 (use chars instead of strings where applicable) - rust-lang#108115 (Do not ICE on unmet trait alias bounds) - rust-lang#108125 (Add new people to the compiletest review rotation) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
@oli-obk and @wesleywiser volunteered to be part of the review rotation of compiletest in https://rust-lang.zulipchat.com/#narrow/stream/238009-t-compiler.2Fmeetings/topic/.5Bweekly.5D.202023-02-16/near/328274017