Add OWNERS files in different subsystems

[0xmichalis]

Already done upstream. This also has come up in discussions with @legionus

@liggitt @deads2k @smarterclayton @ncdc @bparees @mfojtik @pweil- @danmcp

[0xmichalis]

@soltysh @miminar @ironcladlou @derekwaynecarr @pmorie @knobunc

[ncdc]

+1, and also start assigning (automatically via bot?) PRs to the appropriate people for review

[derekwaynecarr]

+1 to both owners and assignee

On Friday, June 10, 2016, Andy Goldstein notifications@github.com wrote:

+1, and also start assigning (automatically via bot?) PRs to the
appropriate people for review

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#9273 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AF8dbFPW8_8Qi1qeL5aBLzWADpL4c7fAks5qKYbNgaJpZM4IzEfK
.

[deads2k]

Are we having trouble getting the right reviewers? I'm not opposed, but I wouldn't want it to be a solution in search of problem. Everyone with merge powers can assign their pull already, right?

[0xmichalis]

@legionus please remind me why you wanted this

On Mon, Jun 13, 2016 at 2:07 PM, David Eads notifications@github.com
wrote:

Are we having trouble getting the right reviewers? I'm not opposed, but I
wouldn't want it to be a solution in search of problem. Everyone with merge
powers can assign their pull already, right?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#9273 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ADuFfy1Nr4s7vsUf0ALpkVf2TGEqNXGYks5qLUfzgaJpZM4IzEfK
.

[smarterclayton]

OWNERS is useful for finding people to nag. Especially for new users.

On Mon, Jun 13, 2016 at 8:12 AM, Michail Kargakis notifications@github.com
wrote:

@Legonius please remind me why you wanted this

On Mon, Jun 13, 2016 at 2:07 PM, David Eads notifications@github.com
wrote:

Are we having trouble getting the right reviewers? I'm not opposed, but I
wouldn't want it to be a solution in search of problem. Everyone with
merge
powers can assign their pull already, right?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9273 (comment)
,
or mute the thread
<
https://github.com/notifications/unsubscribe/ADuFfy1Nr4s7vsUf0ALpkVf2TGEqNXGYks5qLUfzgaJpZM4IzEfK

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#9273 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABG_pw54DI6_sdstNPw1lxOKF5wfY5sYks5qLUlFgaJpZM4IzEfK
.

[soltysh]

+1 on the idea itself and I guess it will be in @stevekuznetsov area :D

[stevekuznetsov]

Since we can do multiple assignments now, do we want to just assign anyone that's an owner of a directory that the PR touches a file under?

[legionus]

I think yes.

[eparis]

I always feel like group assignment means that no one has responsibility...

[eparis]

I'll just believe @pweil- is gonig to hanve this issue and he's going to think that I will....

[stevekuznetsov]

Before we implement anything, first we need to have a conversation on what it means to assign someone a PR. In Origin today we don't even do that. Then I'd like to see a clear set of gains for an OWNERS impl in general and some consensus on what that sort of impl would look like.