Personally, I don't think it matters who merges the pull request. As long as you meet the requirements for code review, then anyone should be able to merge it. In fact, I'd rather have the person who knows most about the change actually merge it into master to ensure that it goes smoothly.
On Fri, Dec 16, 2016 at 12:15 PM, James Sirota <[email protected]> wrote: > Jon, for #2 I changed it to: A committer may merge their own pull request, > but only after a second reviewer has given it a +1. > > 16.12.2016, 10:07, "[email protected]" <[email protected]>: > > I made some minor changes to the doc - check out the history > > <https://cwiki.apache.org/confluence/pages/viewpreviousversions.action? > pageId=61332235> > > if you have any concerns. > > > > Regarding the larger doc - > > 1. Not everybody can assign JIRAs to themselves. I recall I had to > request > > this access, so that should probably be mentioned. > > 2. "A committer may never merge their own pull request, a second party > must > > merge their changes after it has be properly reviewed." > > - Is this still true/accurate? I heard both ways. > > 3. "If somehow the tests get into a failing state on master (such as by a > > backwards incompatible release of a dependency) no pull requests may be > > merged until this is rectified." > > - Maybe this should get reassessed using the > > <https://github.com/apache/incubator-metron/pull/383> most > > <https://github.com/apache/incubator-metron/pull/381> recent > > <https://issues.apache.org/jira/browse/METRON-601> build > > <https://issues.apache.org/jira/browse/METRON-597> failures > > <https://github.com/apache/incubator-metron/pull/380> as a valuable case > > study. > > > > Jon > > > > On Fri, Dec 16, 2016 at 11:38 AM James Sirota <[email protected]> > wrote: > > > >> I threw together a draft document for our release process. Would you > want > >> to add/change/delete anything? > >> > >> ------------------- > >> Thank you, > >> > >> James Sirota > >> PPMC- Apache Metron (Incubating) > >> jsirota AT apache DOT org > > -- > > > > Jon > > > > Sent from my mobile device > > ------------------- > Thank you, > > James Sirota > PPMC- Apache Metron (Incubating) > jsirota AT apache DOT org > -- Nick Allen <[email protected]>
