update docs maintainers #74
Conversation
jenshu
requested review from
a team,
ilevine,
ilrudie,
linsun and
yuval-k
as code owners
| description: Maintainers of the kgateway.dev repo | ||
| maintainers: | ||
| - linsun | ||
| - williamgrh | ||
| members: |
There was a problem hiding this comment.
Did you meam to remove this line?
There was a problem hiding this comment.
no, those are org admins so they need to go under the maintainers section otherwise peribolos complains. everyone under members has maintain privileges in the repo as specified on line 89 below (the naming is confusing)
There was a problem hiding this comment.
Gotcha, so this is members of the maintainer group, as opposed to members of the org.
and maintainers of the maintainers group can do something else?
There was a problem hiding this comment.
Maintainers of a team may "click ops" a user into the team. They really shouldn't because our automation will revert that if the org.yaml doesn't get updated (and it will handle whatever you clicked in with the GUI when org.yaml is updated so it's really just extra work).
Update the org.yaml and maintainers file to reflect the full list of docs maintainers (the yaml was not previously hooked up to the repo so they were out of sync)
Fixes #71