I agree with the fact that ctr can be a very effective way of pushing forward. Ok for me to allow it, it is then to the pr author to consider when he feels that a review is required before merging.
I'm really not used to this way, it will be interesting to see how this performs ;) Andrea On Wed, Nov 16, 2022, 13:31 Bertil Chapuis <bchap...@gmail.com> wrote: > Thanks a lot for all your feedback. > > Josh, your idea is balanced. Opening the PR and tagging reviewers leaves > the opportunity to comment or to object to a change. > > The branch protection rules have been disabled. > > Bertil > > > On 16 Nov 2022, at 09:33, Bertrand Delacretaz <bdelacre...@apache.org> > wrote: > > > > Hi, > > > > Josh Fischer <j...@joshfischer.io> wrote: > >> ...I do find it helpful if I leave a PR open for 12-24 > >> hours to people a chance to make any comments before we merge... > > > > +1, that's a great way of "radiating intent" [1], leaving a chance for > > others to object without slowing down too much. > > > > -Bertrand > > > > [1] > https://medium.com/@ElizAyer/dont-ask-forgiveness-radiate-intent-d36fd22393a3 > > (and also https://davidmarquet.com/turn-the-ship-around-book/ ) > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@baremaps.apache.org > > For additional commands, e-mail: dev-h...@baremaps.apache.org > > > >
