> On 1 Aug 2017, at 10:44 pm, Kai Koehne <kai.koe...@qt.io> wrote: > > I'd like to remind everyone about our commit policy: > https://wiki.qt.io/Review_Policy . In particular, it says 'Give reviewers > ample time to respond', which is assumed to be a full working day minimum > (preferably two days). > > That is, even if you have a +2 by an approver, you should _not_ stage > immediately if there are other reviewers who haven't given feedback yet.
+1 I have often woken up to see changes pushed through over (my) night that I wish I had a chance to comment on, or at least be given the chance to even be aware of some change before it’s been integrated. There also might be reviewers not listed for review that might want to comment. I also understand some P0/urgent changes might need to be rushed through. _______________________________________________ Development mailing list Development@qt-project.org http://lists.qt-project.org/mailman/listinfo/development
