Go code owners: what are primary and secondary owners?

96 views
Skip to first unread message

M Hickford

unread,
Nov 9, 2022, 2:29:00 PM11/9/22
to golan...@googlegroups.com
https://dev.golang.org/owners list primary and secondary owners, but doesn't explain what this means. What do primaries and secondaries do?

Ian Lance Taylor

unread,
Nov 9, 2022, 2:40:01 PM11/9/22
to M Hickford, golan...@googlegroups.com
On Wed, Nov 9, 2022 at 11:28 AM M Hickford <mirth.h...@gmail.com> wrote:
>
> https://dev.golang.org/owners list primary and secondary owners, but doesn't explain what this means. What do primaries and secondaries do?

It's pretty informal. For most packages primary owners are people who
take responsibility for the future direction of the package, including
any major changes. Secondary owners are people who are familiar with
the package and are willing to review bug fixes. But there are no
strict rules or even guidelines.

It's mostly used to add reviewers to incoming Gerritt changes.

Ian

M Hickford

unread,
Nov 9, 2022, 5:11:51 PM11/9/22
to Ian Lance Taylor, M Hickford, golan...@googlegroups.com
Thanks. Does it affect who can approve a change? Or can any Go
maintainer approve any change in any path?
https://go.dev/doc/contribute#votes

M Hickford

unread,
Nov 9, 2022, 5:15:38 PM11/9/22
to M Hickford, Ian Lance Taylor, golan...@googlegroups.com
Presumably all primaries and secondaries are Go maintainers?

Ian Lance Taylor

unread,
Nov 9, 2022, 5:26:10 PM11/9/22
to M Hickford, golan...@googlegroups.com
At present any maintainer (anybody with +2 rights) can approve any
change in any path.

In https://go.dev/issue/53383 we raised the possibility of some people
only having +2 rights over certain files (those that are specific to a
GOOS and/or GOARCH) but that has not been implemented.

Ian

Ian Lance Taylor

unread,
Nov 9, 2022, 5:26:17 PM11/9/22
to M Hickford, golan...@googlegroups.com
On Wed, Nov 9, 2022 at 2:15 PM M Hickford <mirth.h...@gmail.com> wrote:
>
> Presumably all primaries and secondaries are Go maintainers?

Probably? I haven't checked, though.

Ian
Reply all
Reply to author
Forward
0 new messages