<div dir="ltr"><div>Hi All,</div><div><br></div><div dir="ltr">Thanks Alessandro for raising this.<div><br></div><div>First of all, thanks a lot to all PR reviewers for their outstanding work.</div><div><br></div><div>As an occasional QGIS contributor (for a long time), I've definitely noticed a lack of PR reviewers lately. It'd be great if the QGIS association could pay more core devs to participate in the review process.</div><div><br></div><div>Hopefully, that will reduce the number of PRs that go to 'Stale' status, which might be a bit discouraging sometimes.</div><div>This is a topic we discussed with other QGIS contributors in the latest <i>QGIS contributor meeting</i> in Vienna, and they had the same feeling.</div><div><br></div><div><br></div><div>Regards,</div><div><br></div><div>Germán</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El vie, 8 dic 2023 a las 3:35, Alessandro Pasotti via QGIS-Developer (<<a href="mailto:qgis-developer@lists.osgeo.org">qgis-developer@lists.osgeo.org</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, Dec 8, 2023 at 9:22 AM Andreas Neumann <<a href="mailto:andreas@qgis.org" target="_blank">andreas@qgis.org</a>> wrote:<br>
><br>
> Dear Alex,<br>
><br>
> I support this idea. Code reviews are important.<br>
><br>
> But I think we already do that - don't we? In 2023 there was this 14k € budget that is currently being distributed between the core devs who are actively reviewing on Github. Next year, we plan to increase this budget to 20k.<br>
><br>
> Are we missing anything?<br>
<br>
I think the idea was to go from voluntary random PR review<br>
contributions to a regular commitment we can count upon.<br>
<br>
><br>
> I am not sure if it is useful to dedicate an "exact" number of hours and a fixed rate. I'd rather have a fixed annual budget (like we already have) and distribute this among the core devs who actively participate in the reviews and discussions.<br>
<br>
That's exactly the point, some of us think it would be better the other way.<br>
<br>
><br>
> We can formalize this a bit better if you want. I am open for suggestions how to improve the situation.<br>
><br>
> Another idea is to merge the budgets for bug fixing and code reviewing - then the "approved" core devs can spend their hours on both bug fixing and reviewing and invoice everything together.<br>
<br>
No, I would say we keep it separate, I agree that during the<br>
bug-fixing time the participants can spend their time doing both<br>
bugfixing and PR reviews provided that the reviews are oriented to bug<br>
fixing/chores and not to new features.<br>
<br>
><br>
> It is a really good timing to discuss this now - because we want to bring the 2024 budget to vote next week.<br>
<br>
I look forward to other developers' opinions.<br>
<br>
Kind regards.<br>
<br>
><br>
> Greetings,<br>
> Andreas<br>
><br>
> On Fri, 8 Dec 2023 at 08:57, Alessandro Pasotti via QGIS-Developer <<a href="mailto:qgis-developer@lists.osgeo.org" target="_blank">qgis-developer@lists.osgeo.org</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> Recently I have had the opportunity to discuss with some core QGIS<br>
>> developers and we agreed that it would be a good idea if the QGIS<br>
>> organization could fund a few weekly or monthly hours for PR reviews.<br>
>><br>
>> Here is my quick and initial proposal of requisites for the candidates:<br>
>><br>
>> - must be an active well known and long time QGIS C++ core developer<br>
>> (for example from the top last 5 years committers [1])<br>
>> - ideally we'd need more than a single developer: there is not a<br>
>> single developer who is able to review all different areas of the code<br>
>> base<br>
>> - available for regular work (fixed number of weekly or monthly hours,<br>
>> to be determined)<br>
>><br>
>> Note: the above list is just my quick ideas, feel free to suggest<br>
>> anything you think best.<br>
>><br>
>>  I would like this topic to be added to the next PSC meeting agenda.<br>
>><br>
>> Kind regards.<br>
>><br>
>><br>
>> [1] <a href="https://github.com/qgis/QGIS/graphs/contributors?from=2018-12-08&to=2023-12-08&type=c" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/graphs/contributors?from=2018-12-08&to=2023-12-08&type=c</a><br>
>><br>
>> --<br>
>> Alessandro Pasotti<br>
>> QCooperative:  <a href="http://www.qcooperative.net" rel="noreferrer" target="_blank">www.qcooperative.net</a><br>
>> ItOpen:   <a href="http://www.itopen.it" rel="noreferrer" target="_blank">www.itopen.it</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>
><br>
><br>
><br>
> --<br>
><br>
> --<br>
> Andreas Neumann<br>
> <a href="http://QGIS.ORG" rel="noreferrer" target="_blank">QGIS.ORG</a> board member (treasurer)<br>
<br>
<br>
<br>
-- <br>
Alessandro Pasotti<br>
QCooperative:  <a href="http://www.qcooperative.net" rel="noreferrer" target="_blank">www.qcooperative.net</a><br>
ItOpen:   <a href="http://www.itopen.it" rel="noreferrer" target="_blank">www.itopen.it</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><br clear="all"><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>-----------<br></div><div>   |\__  <br>(:>__)(<br>   |/    <br>Soluciones Geoinformáticas Libres                            <br><a href="http://geotux.tuxfamily.org/" target="_blank">http://geotux.tuxfamily.org/</a><br><a href="http://twitter.com/GeoTux2" target="_blank">https://twitter.com/GeoTux2</a></div><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div></div></div></div></div></div></div><div><div><br></div></div></div></div></div></div></div></div>