Skip to content

triagebot: Delete the libs ad-hoc group - #164606

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
tgross35:triagebot-libs
Oct 20, 2026
Merged

rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
tgross35:triagebot-libs

Conversation

@tgross35

Copy link
Copy Markdown
Member

To my understanding, this group was needed prior to the libs team refactor when we had people doing library reviews that weren't actually on T-libs. Since the refactor, everybody here is on T-libs so we should be able to use that default list for reviews. set-team-rotation-mode can be used by anybody who wants to opt out.

r? @Urgau

To my understanding, this group was needed prior to the libs team
refactor when we had people doing library reviews that weren't actually
on T-libs. Since the refactor, everybody here is on T-libs so we should
be able to use that default list for reviews. `set-team-rotation-mode`
can be used by anybody who wants to opt out.
@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 Oct 19, 2026
@tgross35

Copy link
Copy Markdown
Member Author

@Urgau does this make sense? We were a bit confused about why new libs team members weren't getting on the rotation automatically, which I believe this should resolve.

@Urgau

Urgau commented Oct 19, 2026

Copy link
Copy Markdown
Member

I believe so, yes, ad-hoc groups takes precedents over the teams.

Removing the ad-hoc libs group should make everyone in the T-libs team eligible for rotation.
That's what we did for T-compiler in #152326.

cc @Kobzol

@tgross35

Copy link
Copy Markdown
Member Author

That's our understanding as well (discussed on the libs private channel). I'll update our individual members if this looks reasonable

@Kobzol

Kobzol commented Oct 19, 2026

Copy link
Copy Markdown
Member

Indeed this should be possible now. It would be good to check if all original members of the adhoc group are in t-libs (otherwise they would be removed from the rotation by this).

@tgross35

Copy link
Copy Markdown
Member Author

t would be good to check if all original members of the adhoc group are in t-libs (otherwise they would be removed from the rotation by this).

I did check that, everybody is there now 👍 and I think we'd probably just add any users doing libs reviews to T-libs in the future

@tgross35

Copy link
Copy Markdown
Member Author

Would one of you mind approving if this looks reasonable?

@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors r+ rollup

I'll go ahead and approve, I think we have the tools to apply this out of band and/or revert if needed.

@rust-bors

rust-bors Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

📌 Commit b7516ac has been approved by Mark-Simulacrum

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 Oct 20, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 20, 2026
Rollup of 5 pull requests

Successful merges:

 - #164568 (Exclude MaybeDangling like types from SROA)
 - #163771 (update {Ipv4Addr,Ipv6Addr}::is_global to match IANA registries)
 - #164268 ([rustdoc]  Add the Dyn Compatibility heading all the time in the sidebar since it is always in the doc)
 - #164581 (Revert "Remove indirection for `Expr` in `Body`")
 - #164606 (triagebot: Delete the libs ad-hoc group)
@rust-bors
rust-bors Bot merged commit 5d657be into rust-lang:main Oct 20, 2026
14 checks passed
@rustbot rustbot added this to the 1.101.0 milestone Oct 20, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 20, 2026
Rollup merge of #164606 - tgross35:triagebot-libs, r=Mark-Simulacrum

triagebot: Delete the libs ad-hoc group

To my understanding, this group was needed prior to the libs team refactor when we had people doing library reviews that weren't actually on T-libs. Since the refactor, everybody here is on T-libs so we should be able to use that default list for reviews. `set-team-rotation-mode` can be used by anybody who wants to opt out.
@tgross35
tgross35 deleted the triagebot-libs branch October 20, 2026 06:49
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.

5 participants