<div dir="ltr">+1 for all points... sorry to be not so helpful in this period doing my review part for lack of time<div><br><div>I would close some my PRs that I've no time to work on, but I know there are companies interested about that fixes. Would be fair they take care of that. Just matter to setup a appopriate postgis/pki/docker setup to reactivate some PKI tests.</div></div><div><br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">Luigi Pirelli<br><br>**************************************************************************************************<br>* LinkedIn: <a href="https://www.linkedin.com/in/luigipirelli" target="_blank">https://www.linkedin.com/in/luigipirelli</a><br>* Stackexchange: <a href="http://gis.stackexchange.com/users/19667/luigi-pirelli" target="_blank">http://gis.stackexchange.com/users/19667/luigi-pirelli</a><br>* GitHub: <a href="https://github.com/luipir" target="_blank">https://github.com/luipir</a><br>* Mastering QGIS 2nd Edition:<br>* <a href="https://www.packtpub.com/big-data-and-business-intelligence/mastering-qgis-second-edition" target="_blank">https://www.packtpub.com/big-data-and-business-intelligence/mastering-qgis-second-edition</a><br>* Hire me: <a href="http://goo.gl/BYRQKg" target="_blank">http://goo.gl/BYRQKg</a><br>**************************************************************************************************</div></div>
<br><div class="gmail_quote">On 8 May 2018 at 08:26, Alessandro Pasotti <span dir="ltr"><<a href="mailto:apasotti@gmail.com" target="_blank">apasotti@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5"><div class="gmail_extra">On Tue, May 8, 2018 at 1:15 AM, Nyall Dawson <span dir="ltr"><<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
It's no surprise to anyone familiar with the QGIS project that we've<br>
got an issue with the Pull Request queue. It's been slowly growing<br>
over time, recently hitting over 150 open requests! It's a bit of an<br>
embarrassment to the project (some of these PRs have been open for<br>
years!), and is likely causing us to lose new contributors and code.<br>
<br>
The usual magic QGIS coding pixies did some work lately and squashed<br>
the queue back below 100 requests. But the remaining ones are all the<br>
difficult, unfinished or orphaned PRs...<br>
<br>
PR reviewing is hard. Not everyone can review every open PR due to<br>
different familiarity with areas of the codebase. (Which is why I<br>
don't think a funding grant to cover this will ever work<br>
successfully). And no-one wants to be the 'bad guy" who closes an<br>
unmerged PR representing someone else's hard work.<br>
<br>
So I propose a "32 by 3.2" sprint, where we ALL collaboratively aim to<br>
reduce the PR queue to <32 open requests before 3.2 release.<br>
<br>
I think we could achieve this by:<br>
<br>
1. Adopting a hard-line approach to the older, orphaned PRs. Even if<br>
they have some value or reflect real issues, if no-one is interested<br>
in cleaning up the request to get it merge ready then we close it.<br>
<br>
2. Adopt a "open-one, close-one" guideline for core committers. Heck,<br>
I think every core committer has at least 1 or 2 open PRs representing<br>
various experiments and WIP in unfinished states. These should either<br>
be finished off, or closed and re-opened when the work is actually<br>
ready to go. And for test PRs which are "for comment only" I'd suggest<br>
a QEP is more likely to get better feedback and is the more<br>
appropriate place for this discussion of this nature.<br>
<br>
3. Closing orphaned or risky PRs which are targeted to 2.18 and which<br>
have been fixed in master branch.<br>
<br>
4. Sharing the hard work so that the magic pixies don't lose their<br>
magic powers :)<br>
<br>
Thoughts?<br clear="all"></blockquote></div><br><br></div></div></div><div class="gmail_extra">These are all good ideas!<br><br></div><div class="gmail_extra">I'd expecially go for 1, 3 and 4.<span class="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888"><div class="gmail_extra"><br>-- <br><div class="m_6523893902709052658gmail_signature" data-smartmail="gmail_signature">Alessandro Pasotti<br>w3:   <a href="http://www.itopen.it" target="_blank">www.itopen.it</a></div>
</div></font></span></div>
<br>______________________________<wbr>_________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org">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/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br></blockquote></div><br></div>