-
Notifications
You must be signed in to change notification settings - Fork 222
Better define the scope of rustdoc members roles#993
Better define the scope of rustdoc members roles #993GuillaumeGomez wants to merge 2 commits intorust-lang:master from
Conversation
rustbot
commented
Jan 19, 2026
rustbot has assigned @aDotInTheVoid.
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
src/rustdoc/membership.md
Outdated
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.
Do you see anything that needs clarification?
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.
Obvious question is that you haven't mentioned search in any of these. I think it's considered front-end, since it's HTML-specific, but we should probably say so.
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.
Specifying the scope of each team a bit more precisely then.
Kobzol
commented
Jan 19, 2026
I mean, looks sensible, but I'm not sure how is it related to t-infra? 😅
GuillaumeGomez
commented
Jan 19, 2026
For the add of new roles/tags. Also I'm not sure if it works for RFCs yet.
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.
Doesn't this require making a rustdoc-json-backend team (which is probably worthwhile doing anyway)?
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.
Yep, hence why I pinged the infra team. ;)
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.
(Just send a PR to team and that should be it :) )
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.
Dully noted. ;)
Uh oh!
There was an error while loading. Please reload this page.
Since the experiment with the @rust-lang/rustdoc-frontend was a success, I think we can now add the two other roles:
rustdoc-json-backendandrustdoc-internals. That will require for the infra team to also give their opinion here (cc @Kobzol).cc @rust-lang/rustdoc
Rendered