Switch to Code Owners

[reactjs]

Switch to Code Owners

[smikitky]

As an aside, if you want to always assign the same set of reviewers, I think you can just use the code owners functionality. Randomly picking a part of the reviewers is not possible, though.

[tibuurcio]

Hey Nat, I don't quite understand the problem. How is the sync script doing it? And how it would do to assign maintainers without GitHub's API (Do you mean GitHub's UI)?

[halian-vilela]

Hey Nat, I don't quite understand the problem. How is the sync script doing it? And how it would do to assign maintainers without GitHub's API (Do you mean GitHub's UI)?

Me neither, could you explain the issue a little bit further?

[tesseralis]

@tibuurcio @halian-vilela I just mean that when I create a sync pull request, I want to automatically assign "Reviewers" to you or the other pt-br maintainers. Since there are a lot of you, I want to do it in a way that makes sense. If I assign all of you then that might result in duplicated work. I was thinking of assigning the issue randomly to 2-3 maintainers.

[halian-vilela]

@tibuurcio @halian-vilela I just mean that when I create a sync pull request, I want to automatically assign "Reviewers" to you or the other pt-br maintainers. Since there are a lot of you, I want to do it in a way that makes sense. If I assign all of you then that might result in duplicated work. I was thinking of assigning the issue randomly to 2-3 maintainers.

@tesseralis

Got it.. randomly to 2 maintainers seems reasonable to me.

[smikitky]

Randomly assigning two of the three maintainers may not make much sense at least in the Japanese fork. We check and approve each sync PR on a first-come basis. It should be either assigning everyone or assigning no one.

[QC-L]

I also think we should assigning no one.

[tesseralis]

@QC-L could you elaborate on why?

[QC-L]

Random assignment does not necessarily mean someone will respond.
I refresh repo every day, so I will merge when I see it.
I assigned tasks to some people, but you may not receive mail.

[QC-L]

@tesseralis I agree to use code owners.