<div dir="ltr"><div>Dear Alex,</div><div><br></div><div>I support this idea. Code reviews are important.<br></div><div><br></div><div>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.</div><div><br></div><div>Are we missing anything?<br></div><div><br></div><div>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.</div><div><br></div><div>We can formalize this a bit better if you want. I am open for suggestions how to improve the situation.</div><div><br></div><div>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.</div><div><br></div><div>It is a really good timing to discuss this now - because we want to bring the 2024 budget to vote next week.<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 Fri, 8 Dec 2023 at 08:57, Alessandro Pasotti via QGIS-Developer <<a href="mailto:qgis-developer@lists.osgeo.org">qgis-developer@lists.osgeo.org</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,<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>
</blockquote></div><br clear="all"><br><span class="gmail_signature_prefix">-- </span><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>