Remove rustdoc adhoc group#152310
Conversation
|
|
|
r? @jieyouxu rustbot has assigned @jieyouxu. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
r? Urgau |
|
I set everyone except GuillaumeGomez, fmease and notriddle to be |
|
The other members are off rotation because of "Team review prefs". |
|
@bors r+ rollup |
… r=Urgau Remove `rustdoc` adhoc group With rust-lang/triagebot#2273, reviewers can now tell triagebot to opt them out of being assigned through a given team, so that `r? <team>` does not consider them if they don't want to. That means that we can now finally get rid of adhoc groups that correspond to actual Rust teams. I wanted to start with a smaller team than `t-compiler` to test this, so I started with `rustdoc`, which was the smallest team that I found an adhoc group for. Currently, the `rustdoc` team contains the following members: - aDotInTheVoid - GuillaumeGomez - notriddle - fmease - camelid - Manishearth - Urgau - lolbinarycat - yotamofek Only three of those were previously assignable through `r? rustdoc`. So to reproduce the previous state, I will run `as <username> work set-team-rotation-mode rustdoc off` for the remaining members of the `rustdoc` team before this is merged. If more people from `t-rustdoc` want to join the review rotation, they can send [triagebot](https://rust-lang.zulipchat.com/#narrow/dm/261224-triagebot) a DM with `work set-team-rotation-mode rustdoc on`. CC @rust-lang/rustdoc
Remove `rustdoc` adhoc group With rust-lang/triagebot#2273, reviewers can now tell triagebot to opt them out of being assigned through a given team, so that `r? <team>` does not consider them if they don't want to. That means that we can now finally get rid of adhoc groups that correspond to actual Rust teams. I wanted to start with a smaller team than `t-compiler` to test this, so I started with `rustdoc`, which was the smallest team that I found an adhoc group for. Currently, the `rustdoc` team contains the following members: - aDotInTheVoid - GuillaumeGomez - notriddle - fmease - camelid - Manishearth - Urgau - lolbinarycat - yotamofek Only three of those were previously assignable through `r? rustdoc`. So to reproduce the previous state, I will run `as <username> work set-team-rotation-mode rustdoc off` for the remaining members of the `rustdoc` team before this is merged. If more people from `t-rustdoc` want to join the review rotation, they can send [triagebot](https://rust-lang.zulipchat.com/#narrow/dm/261224-triagebot) a DM with `work set-team-rotation-mode rustdoc on`. CC @rust-lang/rustdoc
|
@bors yield (to enclosing rollup) |
|
Auto build cancelled. Cancelled workflows: The next pull request likely to be tested is #152314. |
Rollup merge of #152310 - Kobzol:remove-rustdoc-adhoc-group, r=Urgau Remove `rustdoc` adhoc group With rust-lang/triagebot#2273, reviewers can now tell triagebot to opt them out of being assigned through a given team, so that `r? <team>` does not consider them if they don't want to. That means that we can now finally get rid of adhoc groups that correspond to actual Rust teams. I wanted to start with a smaller team than `t-compiler` to test this, so I started with `rustdoc`, which was the smallest team that I found an adhoc group for. Currently, the `rustdoc` team contains the following members: - aDotInTheVoid - GuillaumeGomez - notriddle - fmease - camelid - Manishearth - Urgau - lolbinarycat - yotamofek Only three of those were previously assignable through `r? rustdoc`. So to reproduce the previous state, I will run `as <username> work set-team-rotation-mode rustdoc off` for the remaining members of the `rustdoc` team before this is merged. If more people from `t-rustdoc` want to join the review rotation, they can send [triagebot](https://rust-lang.zulipchat.com/#narrow/dm/261224-triagebot) a DM with `work set-team-rotation-mode rustdoc on`. CC @rust-lang/rustdoc
…-Simulacrum Remove more adhoc groups that correspond to teams Following in the footsteps of rust-lang#152310. - [types](https://github.com/rust-lang/team/blob/main/teams/types.toml) team - removed Niko and lqd from team rotation in triagebot - [project-const-traits](https://github.com/rust-lang/team/blob/main/teams/project-const-traits.toml) - no changes needed, whole team was already on rotation - [project-stable-mir](https://github.com/rust-lang/team/blob/main/teams/project-stable-mir.toml) - no changes needed, whole team was already on rotation - [project-exploit-mitigations](https://github.com/rust-lang/team/blob/main/teams/project-exploit-mitigations.toml) - removed 1c3t3a from team rotation in triagebot (TODO - it didn't work yet, need to fix triagebot to allow running commands on top of people that are only in project/working groups)
Rollup merge of #152325 - Kobzol:remove-adhoc-groups, r=Mark-Simulacrum Remove more adhoc groups that correspond to teams Following in the footsteps of #152310. - [types](https://github.com/rust-lang/team/blob/main/teams/types.toml) team - removed Niko and lqd from team rotation in triagebot - [project-const-traits](https://github.com/rust-lang/team/blob/main/teams/project-const-traits.toml) - no changes needed, whole team was already on rotation - [project-stable-mir](https://github.com/rust-lang/team/blob/main/teams/project-stable-mir.toml) - no changes needed, whole team was already on rotation - [project-exploit-mitigations](https://github.com/rust-lang/team/blob/main/teams/project-exploit-mitigations.toml) - removed 1c3t3a from team rotation in triagebot (TODO - it didn't work yet, need to fix triagebot to allow running commands on top of people that are only in project/working groups)
With rust-lang/triagebot#2273, reviewers can now tell triagebot to opt them out of being assigned through a given team, so that
r? <team>does not consider them if they don't want to.That means that we can now finally get rid of adhoc groups that correspond to actual Rust teams. I wanted to start with a smaller team than
t-compilerto test this, so I started withrustdoc, which was the smallest team that I found an adhoc group for.Currently, the
rustdocteam contains the following members:Only three of those were previously assignable through
r? rustdoc. So to reproduce the previous state, I will runas <username> work set-team-rotation-mode rustdoc offfor the remaining members of therustdocteam before this is merged.If more people from
t-rustdocwant to join the review rotation, they can send triagebot a DM withwork set-team-rotation-mode rustdoc on.CC @rust-lang/rustdoc