Skip to content

Remove rustdoc adhoc group#152310

Merged
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
Kobzol:remove-rustdoc-adhoc-group
Feb 8, 2026
Merged

Remove rustdoc adhoc group#152310
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
Kobzol:remove-rustdoc-adhoc-group

Conversation

@Kobzol
Copy link
Member

@Kobzol Kobzol commented Feb 7, 2026

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 a DM with work set-team-rotation-mode rustdoc on.

CC @rust-lang/rustdoc

@rustbot
Copy link
Collaborator

rustbot commented Feb 7, 2026

triagebot.toml has been modified, there may have been changes to the review queue.

cc @davidtwco, @wesleywiser

@rustbot rustbot added A-meta Area: Issues & PRs about the rust-lang/rust repository itself S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 7, 2026
@rustbot
Copy link
Collaborator

rustbot commented Feb 7, 2026

r? @jieyouxu

rustbot has assigned @jieyouxu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Fallback group: @Mark-Simulacrum, @jieyouxu
  • @Mark-Simulacrum, @jieyouxu expanded to Mark-Simulacrum, jieyouxu
  • Random selection from Mark-Simulacrum, jieyouxu

@Urgau
Copy link
Member

Urgau commented Feb 7, 2026

r? Urgau

@rustbot rustbot assigned Urgau and unassigned jieyouxu Feb 7, 2026
@Kobzol
Copy link
Member Author

Kobzol commented Feb 7, 2026

I set everyone except GuillaumeGomez, fmease and notriddle to be off rotation for rustdoc, to reproduce the earlier state. You can check by sending team-stats rustdoc to triagebot on Zulip.

@Urgau
Copy link
Member

Urgau commented Feb 7, 2026

team-stats rustdoc reports as expected GuillaumeGomez, notriddle and fmease as being on-rotation.

The other members are off rotation because of "Team review prefs".

Copy link
Member

@Urgau Urgau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Let's try it.

View changes since this review

@Urgau
Copy link
Member

Urgau commented Feb 7, 2026

@bors r+ rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Feb 7, 2026

📌 Commit d53133e has been approved by Urgau

It is now in the queue for this repository.

@rust-bors rust-bors bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 7, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Feb 7, 2026
… 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
rust-bors bot pushed a commit that referenced this pull request Feb 8, 2026
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
@rust-bors
Copy link
Contributor

rust-bors bot commented Feb 8, 2026

⌛ Testing commit d53133e with merge 90bc0f9...

@Zalathar
Copy link
Member

Zalathar commented Feb 8, 2026

@bors yield (to enclosing rollup)

@rust-bors
Copy link
Contributor

rust-bors bot commented Feb 8, 2026

Auto build cancelled. Cancelled workflows:

The next pull request likely to be tested is #152314.

rust-bors bot pushed a commit that referenced this pull request Feb 8, 2026
Rollup of 2 pull requests

Successful merges:

 - #152310 (Remove `rustdoc` adhoc group)
 - #152284 (Avoid a bogus THIR span for `let x = offset_of!(..)`)
@rust-bors rust-bors bot merged commit d016a67 into rust-lang:main Feb 8, 2026
11 of 12 checks passed
@rustbot rustbot added this to the 1.95.0 milestone Feb 8, 2026
rust-timer added a commit that referenced this pull request Feb 8, 2026
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
@Kobzol Kobzol deleted the remove-rustdoc-adhoc-group branch February 8, 2026 07:16
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Feb 8, 2026
…-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)
rust-timer added a commit that referenced this pull request Feb 9, 2026
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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-meta Area: Issues & PRs about the rust-lang/rust repository itself S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants