On 2018-04-13 06:00 AM, Daniel Vetter wrote: > This tries to align with the X.org communities's long-standing > tradition of trying to be an inclusive community and handing out > commit rights fairly freely. > > We also tend to not revoke commit rights for people no longer > regularly active in a given project, as long as they're still part of > the larger community. > > Finally make sure that commit rights, like anything happening on fd.o > infrastructre, is subject to the fd.o's Code of Conduct. > > v2: Point at MAINTAINERS for contact info (Daniel S.) > > v3: > - Make it clear that commit rights are voluntary and that committers > need to acknowledge positively when they're nominated by someone > else (Keith). > - Encourage committers to drop their commit rights when they're no > longer active, and make it clear they'll get readded (Keith). > - Add a line that maintainers and committers should actively nominate > new committers (me). > > v4: Typo (Petri). > > v5: Typo (Sean). > > v6: Wording clarifications and spelling (Jani). > > v7: Require an explicit commitment to the documented merge criteria > and rules, instead of just the implied one through the Code of Conduct > threat (Jani). > > Acked-by: Alex Deucher <alexander.deucher@xxxxxxx> > Acked-by: Arkadiusz Hiler <arkadiusz.hiler@xxxxxxxxx> > Acked-by: Daniel Stone <daniel@xxxxxxxxxxxxx> > Acked-by: Eric Anholt <eric@xxxxxxxxxx> > Acked-by: Gustavo Padovan <gustavo@xxxxxxxxxxx> > Acked-by: Petri Latvala <petri.latvala@xxxxxxxxx> Acked-by: Harry Wentland <harry.wentland@xxxxxxx> Harry > Cc: Alex Deucher <alexander.deucher@xxxxxxx> > Cc: Arkadiusz Hiler <arkadiusz.hiler@xxxxxxxxx> > Cc: Ben Widawsky <ben@xxxxxxxxxxxx> > Cc: Daniel Stone <daniel@xxxxxxxxxxxxx> > Cc: Dave Airlie <airlied@xxxxxxxxx> > Cc: Eric Anholt <eric@xxxxxxxxxx> > Cc: Gustavo Padovan <gustavo@xxxxxxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > Cc: Keith Packard <keithp@xxxxxxxxxx> > Cc: Kenneth Graunke <kenneth@xxxxxxxxxxxxx> > Cc: Kristian H. Kristensen <hoegsberg@xxxxxxxxxx> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Cc: Petri Latvala <petri.latvala@xxxxxxxxx> > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > Cc: Sean Paul <seanpaul@xxxxxxxxxxxx> > Reviewed-by: Keith Packard <keithp@xxxxxxxxxx> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > --- > If you wonder about the wide distribution list for an igt patch: I'd > like to start a discussions about x.org community norms around commit > rights at large, at least for all the shared repos. I plan to propose > the same text for drm-misc and libdrm too, and hopefully others like > mesa/xserver/wayland would follow. > > fd.o admins also plan to discuss this (and a pile of other topics and > hosting and code of conduct) with all projects, ideally this here > would end up as the starting point for establishing some community > norms. > -Daniel > --- > CONTRIBUTING | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 48 insertions(+) > > diff --git a/CONTRIBUTING b/CONTRIBUTING > index 0180641be3aa..8a118134275c 100644 > --- a/CONTRIBUTING > +++ b/CONTRIBUTING > @@ -51,4 +51,52 @@ A short list of contribution guidelines: > - Changes to the testcases are automatically tested. Take the results into > account before merging. > > +Commit rights > +------------- > + > +Commit rights will be granted to anyone who requests them and fulfills the > +below criteria: > + > +- Submitted a few (5-10 as a rule of thumb) non-trivial (not just simple > + spelling fixes and whitespace adjustment) patches that have been merged > + already. > + > +- Are actively participating on discussions about their work (on the mailing > + list or IRC). This should not be interpreted as a requirement to review other > + peoples patches but just make sure that patch submission isn't one-way > + communication. Cross-review is still highly encouraged. > + > +- Will be regularly contributing further patches. This includes regular > + contributors to other parts of the open source graphics stack who only > + do the oddball rare patch within igt itself. > + > +- Agrees to use their commit rights in accordance with the documented merge > + criteria, tools, and processes. > + > +Apply for an account (and any other account change requests) through > + > +https://www.freedesktop.org/wiki/AccountRequests/ > + > +and please ping the maintainers if your request is stuck. > + > +Committers are encouraged to request their commit rights get removed when they > +no longer contribute to the project. Commit rights will be reinstated when they > +come back to the project. > + > +Maintainers and committers should encourage contributors to request commit > +rights, especially junior contributors tend to underestimate their skills. > + > +Code of Conduct > +--------------- > + > +Please be aware the fd.o Code of Conduct also applies to igt: > + > +https://www.freedesktop.org/wiki/CodeOfConduct/ > + > +See the MAINTAINERS file for contact details of the igt maintainers. > + > +Abuse of commit rights, like engaging in commit fights or willfully pushing > +patches that violate the documented merge criteria, will also be handled through > +the Code of Conduct enforcement process. > + > Happy hacking! > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel