<div dir="ltr"><div>Hi Nyall,</div><div><br></div><div>Thanks for raising this issue!</div><div><br></div><div>Back in the "old days" we had assigned "maintainers" to different areas of the code base. Maybe we could revive this idea but rename it to "reviewers" and ideally assign more than one person to each area of expertise? It could be published again, somewhere in the governance section of our website, e.g. at <a href="https://www.qgis.org/en/site/getinvolved/governance/governance.html">https://www.qgis.org/en/site/getinvolved/governance/governance.html</a></div><div><br></div><div>As a side effect, if we had such a list, a potential customer could also see - oh this developer is a reviewer/maintainer of this particular topic - it could be an incentive to pick this developer or company for development work. For some insiders, it is a well-known which dev/company has expertise in which area of QGIS, but for non-insiders, this is not really obvious.<br></div><div><br></div><div>Ideally, besides the volunteers, each of the companies that is active in QGIS development (like Lutra, Oslandia, OPENGIS, 3Liz, etc. - just to name a few) could offer one person as a reviewer to a certain area in the code base. Of course, some, such as Norbit and Lutra are already really active in packaging QGIS ... and North Road and OPENGIS is already quite active in reviewing and other maintenance work.<br></div><div><br></div><div>Greetings,</div><div>Andreas<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 23 Mar 2021 at 00:06, Nyall Dawson <<a href="mailto:nyall.dawson@gmail.com">nyall.dawson@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi list,<br>
<br>
This is a public plea for more developers who are very familiar with<br>
different parts of the QGIS codebase to become actively involved in<br>
backport PR management.<br>
<br>
We NEED more people to be actively (i.e. daily) monitoring for<br>
backport PRs and then reviewing them if the backport affects code<br>
areas which they're knowledgeable about.<br>
<br>
Right now there's like... 2... of us doing this on a daily basis, and<br>
we need more hands and eyes. Regressions are fixed by backports, but<br>
they can also be introduced by backports. If we don't get more people<br>
involved in this part of QGIS' maintenance then the stability of the<br>
patch releases is compromised.<br>
<br>
While I'm more than happy to keep reviewing backports which touch<br>
areas of code I know well, we have a whole stack of backports for<br>
server fixes, relation fixes, and other parts of QGIS I don't know<br>
well enough to risk release stability by reviewing myself. These<br>
backports are currently sitting in the queue for weeks without review,<br>
and often they stale out and are closed without ever getting merged.<br>
<br>
I've also a ton of backports for fixes that I make on a daily basis<br>
which I cannot self review, so the PR queue gets flooded with my<br>
backports until I actively pester/harass others to review and approve.<br>
<br>
The truth is that the longer the PR queue, the more it stresses me out<br>
and the less motivation I have to review the incoming feature PRs. We<br>
have many older feature PRs which are stalled and waiting review for<br>
months or longer, just because I'm so flooded with managing newer<br>
incoming PRs that I never have time to review the older ones...<br>
<br>
So please, if you know parts of the QGIS code well, and can spare 15<br>
minutes a day to looking over the backport queue and approving any<br>
backports you are confident are regression free, then please get<br>
involved! (or ask your employer to officially allocate you this 15<br>
minutes/day as a way of them supporting QGIS!)<br>
<br>
Nyall<br>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><br>--<br>Andreas Neumann<br></div><a href="http://QGIS.ORG" target="_blank">QGIS.ORG</a> board member (treasurer)<br></div></div>