Tool for generating/maintaining/auditing CODEOWNER file about automation HOT 12 CLOSED

For instance, ncu could include a git command that uses the CODEOWNERS file to identify who would be notified for any given set of commits.

I am curious, which commands could use that?
I thought about something for the release team to find proper backporters but have not reached the point where I can start writing code yet, but the owners specified by the CODEOWNERS file seems to be too broad in that case, at least not better than the "reviewer + author" group.

from automation.

@jasnell can you point to any doc or the existing CODEOWNERS files? I touch a quick look in the src directory but I did not see it.

from automation.

https://github.com/nodejs/node/blob/master/.github/CODEOWNERS

from automation.

@BridgeAR thanks !

from automation.

Currently, the CODEOWNERS file is of limited value because it will only do automatic notifications if the team or individuals have write access in the repository. Since most o our teams include people who do not have write access, GitHub does not automatically process the notifications. This makes me sad.

We could augment that by having the Github bot automatically do those notifications for us based on CODEOWNERS file. It's a stop gap measure, but a useful one.

Another idea would be having git node land use the CODEOWNERS file to determine if anyone from the owning teams has reviewed the PR in question. That would, of course, require the git nod land tool to know who is on each team, but, for instance, if the PR changes http2 and no one from the nodejs/http2 has reviewed, then the landing tool can emit a warning.

from automation.

@jasnell Maybe we should replace the teams in CODEOWNERS with teams that have write access? Otherwise it does not seem to be useful to have them anyway.

from automation.

By the way ncu already has code (for ncu-team) to know about members in teams and requires necessary access when creating tokens.

from automation.

I was wondering. If we're overriding GitHub, then we already have the bot doing it's black magic for tags. IMHO we should reuse that for @ mentioning teams.

from automation.

Along the lines of what @refack mentioned, would it be easier, for now, to avoid CODEOWNERS and handle all of this in github-bot? It already matches file paths to add labels to PRs, in theory we could do the same thing but just match to the relevant team(s) to ping. For example, if a PR edits http2 code (it will currently get the http2 label), and we could request a review from the http2 GitHub team.

from automation.

So my only question there is how we would best manage those mappings in the GitHub bot. Otherwise tho, I'm perfectly fine with that approach so long as it works :)

from automation.

I think we just need to make sure that the labels match the team names?

from automation.

Unarchived this repo so I could close all the PRs and issues. Will re-archive when I'm done.

from automation.