Not really a decision, just it feels safer to me personally to have someone looking at them before I merge them.
But, sure, if there's another way, that's fine too. Thanks as always Bertrand for your insights and guidance. :-) Gj On Fri, Nov 3, 2017 at 9:35 AM, Bertrand Delacretaz <[email protected]> wrote: > Hi, > > On Fri, Nov 3, 2017 at 1:11 AM, Geertjan Wielenga > <[email protected]> wrote: > > ...Several more pull requests now -- people are needed to review them... > > I'm curious why you want reviews as opposed to just merging those yourself. > > I see mostly trivial changes at > https://github.com/apache/incubator-netbeans/pulls > > Was there a decision to use Review-Then-Commit instead of > Commit-Then-Review? > > And if yes where's that documented? > > -Bertrand (incubation mentor hat on - trying to avoid making things > more complicated than needed) >
