<div dir="ltr"><div>Hi Sandro,</div><div><br></div><div>We have discussed this proposal today and we think that this is a good idea.</div><div><br></div><div>Starting with the next bug fixing round the developers who participate will be allowed to use the paid bug fixing time to do PR reviews of the bug fixes done by other developers.</div><div><br></div><div>It is probably simpler to add a new "Reviewed by" column to the spreadsheet than having multiple rows.<br></div><div><br></div><div>Thank you for this proposal.</div><div><br></div><div>Kind regards.<br></div><div> <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 16, 2022 at 3:36 PM Sandro Santilli <<a href="mailto:strk@kbt.io">strk@kbt.io</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">Having partecipated in a few bugfixing campaigns I'm writing this<br>
email to propose an improvement in the campaign, which suffers from<br>
the hard-to-obtain reviews on pull requests.<br>
<br>
The problem: it is hard to get reviews on pull requests, so they may<br>
be stuck in "need review" state for a long time.<br>
There are currently 54 such PRs:<br>
<a href="https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aopen+review%3Arequired" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aopen+review%3Arequired</a><br>
<br>
Suggested solution: allow logging _reviews_ on the sheet used for<br>
reporting work performed by partecipating developers:<br>
<a href="https://docs.google.com/spreadsheets/d/1F6v4g8Ayb3wIt73rxFKD8h4B95MaTwt-eDit8YvReB0/edit#gid=1300379589" rel="noreferrer" target="_blank">https://docs.google.com/spreadsheets/d/1F6v4g8Ayb3wIt73rxFKD8h4B95MaTwt-eDit8YvReB0/edit#gid=1300379589</a><br>
<br>
At the moment the sheet contains a line for each task taken by<br>
each developer. We could add an additional column to report<br>
the name of the reviewer (to be filled by the PR author) or we<br>
could have a column to express whether the developer only did<br>
the review of a PR done by someone else.<br>
<br>
In the first case each PR has only one line, in the second case<br>
each PR will possibly have multiple lines.<br>
<br>
What do you think ?<br>
<br>
--strk; <br>
<br>
  Libre GIS consultant/developer<br>
  <a href="https://strk.kbt.io/services.html" rel="noreferrer" target="_blank">https://strk.kbt.io/services.html</a><br>
_______________________________________________<br>
Qgis-psc mailing list<br>
<a href="mailto:Qgis-psc@lists.osgeo.org" target="_blank">Qgis-psc@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/qgis-psc" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-psc</a><br>
</blockquote></div>