highfive icon indicating copy to clipboard operation
highfive copied to clipboard

Assigning T-team labels when highfive assigns a reviewer

Open pietroalbini opened this issue 7 years ago • 3 comments

It would be awesome if highfive, after choosing a reviewer from a team, assigned the T-team label of that team. This will allow scripts to easily see the assigned team, for example to build graphs.

pietroalbini avatar May 03 '18 10:05 pietroalbini

Is there a way to query which team a user is on or are you imagining that we would encode that in the repo config file? The rust config has several groups (compiler, syntax, libs). Do those align with teams? If yes, are there situations where you wouldn't want them to align with teams? If that is no, we could talk about changing the config format to optionally specify labels for each group.

davidalber avatar May 03 '18 15:05 davidalber

Is there a way to query which team a user is on or are you imagining that we would encode that in the repo config file?

Some people are in more than one team, so the config file would be better.

The rust config has several groups (compiler, syntax, libs). Do those align with teams?

Both compiler and syntax are T-compiler, and libs is T-libs. From _global.json doc is T-doc.

If yes, are there situations where you wouldn't want them to align with teams?

Technically yes, if someone only changes the docstring of a stdlib method the PR is assigned to the libs team, but that's not a new problem, the reviewer is also wrongly assigned, so I wouldn't worry about it.

If that is no, we could talk about changing the config format to optionally specify labels for each group.

That would be awesome :)

pietroalbini avatar May 03 '18 15:05 pietroalbini

I don't think you can choose a team based on the reviewer because of people belonging to multiple teams (for which team.html on r-l.o is the source of truth). But putting a label based on the config sounds good.

nrc avatar May 03 '18 19:05 nrc