<div dir="ltr"><div dir="ltr"><div dir="ltr">Hi, <div><br></div><div>First of all, thanks Nyall for your efforts. I try to review the code that we introduced to QGIS and I find kind-of familiar: for example QgsQuick, MeshLayer or MacOS stuff. But as mentioned in the thread, the less you know the particular code, the more time it takes to review it.  What I see that I can improve is to actively at least twice per week double check the PR queue to see if there is something in this area, since sometimes I need to be pinged to get a notification in email that the PR is waiting for review. </div><div><br></div><div>What are my thoughts on other improvements not mentioned by Nyall and others:</div><div>1. when someone introduces a new feature (and most notably coming from a paid contract), the review process should be taken into account in the quote and the reviewer should be notified. Proper review can take days...<br></div><div>2.  we may have a (partially paid?) role for the review manager, to go through the PR queue, ping people for reviews, etc. (Alternatively) <span style="color:rgb(0,0,0);font-family:-webkit-standard">create some github-action bot to assign the maintainers to the PRs as reviewers?</span></div><div>3. reintroduce the maintainers of particular QGIS code-base. For example here the motivation for companies could be to advertise their logo in the "sustainable member section"  or introducing some other category to advertise their efforts.</div><div><br></div><div>Cheers,</div><div>Peter</div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 9, 2021 at 7:45 AM 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-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On Thu, 8 Apr 2021 at 18:26, Julien Cabieces<br>
<<a href="mailto:julien.cabieces@oslandia.com" target="_blank">julien.cabieces@oslandia.com</a>> wrote:<br>
><br>
><br>
> Hi Nyall,<br>
><br>
> We (at Oslandia) heard your call for more volunteers in reviewing the backports and PRs.<br>
><br>
> We are very well aware that our contribution effort for this task is not enough. I see mainly two reasons:<br>
> - Lack of time and human ressources to fullfill this task. This is a bad reason, and a well known situation that we need to take care of internally, to free our schedule.<br>
> - A very partial knowledge of the code base in which we feel relevant to review. I try myself to review all Oracle related PRs, or take a look in some part of the code I know enough to make a good review,<br>
<br>
It would certainly be remiss of me not to acknowledge your hard work<br>
in maintaining the Oracle provider over recent months -- it's<br>
certainly very highly valued!<br>
<br>
> but it's very little compared to the QGIS code base.<br>
<br>
This is where things get really tricky. We do have a very small pool<br>
of people with the required expertise to handle the reviewing process,<br>
and it's a very fine balancing act of "opening this up to a wider<br>
community" vs "ensure that nothing nasty gets through". I don't have<br>
any easy answers here, but a few random thoughts:<br>
<br>
- ANY review is appreciated, even if it's just a "I had a look at this<br>
and the c++ code looks good to me, but I'm not familiar with the QGIS<br>
raster api so can someone else check this part out".<br>
- There's a lot of "chore" work in the pr review process which has a<br>
lower barrier of entry. For instance guiding new contributors through<br>
the QGIS processes such as the formatting scripts and CI setup. We<br>
regularly get PRs from first-time contributors and they almost always<br>
need some mentoring in the QGIS processes before they pass the CI<br>
setup and are ready for an in-depth review. This is the kind of thing<br>
which we should ideally do ASAP, as the faster we get the contributors<br>
through the initial maze of QGIS contributions the more likely they<br>
are to continue to contribute.<br>
- We've discussed this before, but a quick comment from a<br>
non-developer on new contributors PRs to say "great work, welcome<br>
abroad, this sounds like a very valuable contribution" is enough to<br>
give the project a positive vibe and give motivation to the submitter<br>
to get their PR in shape to merge<br>
- PR reviewing is a great way to learn new parts of the QGIS codebase ;)<br>
<br>
> I hesitated answering your first email, saying basically that we will try to improve the situation, but it would have been just a promise, and I'm not sure that we would have been able to keep it, regarding the points I described above.<br>
<br>
I really appreciate the reply, and the time you've taken to drop your<br>
thoughts into this conversation!<br>
<br>
Nyall<br>
<br>
<br>
<br>
<br>
><br>
><br>
> Kind regard,<br>
> Julien<br>
><br>
> > On Tue, 23 Mar 2021 at 09:06, Nyall Dawson <<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>> wrote:<br>
> >><br>
> >> 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>
> ><br>
> > Well my earlier plea seems like it fell on deaf ears, and we've seen<br>
> > only minimal assistance coming to maintain the PR queue since I posted<br>
> > this.<br>
> ><br>
> > I'll ask nicely once more. If you're an organisation involved in QGIS<br>
> > development, you NEED to donate time to maintaining this list. It's<br>
> > not enough to just ensure your own PRs get through the queue, you HAVE<br>
> > to help with the shared burden of getting others PRs reviewed and<br>
> > merged.<br>
> ><br>
> > This callout applies to ALL organisations who are making money from<br>
> > core QGIS development. You know who you are, and you can ALL afford to<br>
> > donate 30 minutes of a staff member's time each week to helping with<br>
> > this shared burden. If your boss is blocking this then you NEED to let<br>
> > them know how big a risk they are running here. It's not fair to leave<br>
> > this responsibility on the already burdened shoulders of a few<br>
> > overworked individuals.<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>
><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>
_______________________________________________<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></div>