diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2017-07-18 18:00:20 +0200 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2017-07-20 10:49:27 +0200 |
commit | c42ab4d23f4bafecf51574f538db8f5142519523 (patch) | |
tree | e178451d01ba4eee823f0c924f9d3617216621df /configure.ac | |
parent | a865fee5bbfa573823f758b85116e00934083f93 (diff) |
CONTRIBUTING: formalize review rules
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 <daniels@collabora.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Acked-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Acked-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Acked-by: Petri Latvala <petri.latvala@intel.com>
Acked-by: Imre Deak <imre.deak@intel.com>
Acked-by: Robert Foss <robert.foss@collabora.com>
Acked-by: Ben Widawsky <ben@bwidawsk.net>
Acked-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Acked-by: Mika Kuoppala <mika.kuoppala@intel.com>
Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Acked-by: Eric Anholt <eric@anholt.net>
Acked-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Acked-by: Lyude <lyude@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'configure.ac')
0 files changed, 0 insertions, 0 deletions