On Wed, Jul 19, 2017 at 11:03:46AM +0300, Arkadiusz Hiler wrote: > On Tue, Jul 18, 2017 at 06:00:20PM +0200, Daniel Vetter wrote: > > There's a bunch of reasons why I think we should formalize and enforce > > our review rules for igt patches: > > > > - We have a lot of new engineers joining and review helps enormously > > with mentoring and learning. But right now only patches from > > contributors without commit rights are consistently subjected to > > review, which makes this imbalanced and removes senior contributors > > from the review pool. > > > > - We have a much bigger team and we need to make sure we're aligned on > > where igt as a tool and testsuite needs to head towards. Getting > > that alignment happens through reviewing each other's submission. > > Pushing a contentious patch and then dealing with a heated irc > > discussion is much less effective. > > > > - Finally igt becomes ever more important for our testing, making sure > > the code quality is high is important. Review helps with that. > > > > v2: Improve wording a bit (Imre). > > > > Acked-by: Daniel Stone <[email protected]> > > Acked-by: Jani Nikula <[email protected]> > > Acked-by: Joonas Lahtinen <[email protected]> > > Acked-by: Maarten Lankhorst <[email protected]> > > Acked-by: Petri Latvala <[email protected]> > > Acked-by: Imre Deak <[email protected]> > > Acked-by: Robert Foss <[email protected]> > > Acked-by: Ben Widawsky <[email protected]> > > Acked-by: Tvrtko Ursulin <[email protected]> > > Acked-by: Mika Kuoppala <[email protected]> > > Signed-off-by: Daniel Vetter <[email protected]> > Acked-by: Arkadiusz Hiler <[email protected]> > > With the recent growth in contributions it's a good thing :-)
And pushed, thx everyone for their acks. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
