[Qgis-psc] Payed review time

[Alessandro Pasotti]

Hi Sandro,

We have discussed this proposal today and we think that this is a good idea.

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.

It is probably simpler to add a new "Reviewed by" column to the spreadsheet
than having multiple rows.

Thank you for this proposal.

Kind regards.


On Wed, Feb 16, 2022 at 3:36 PM Sandro Santilli <strk at kbt.io> wrote:

> Having partecipated in a few bugfixing campaigns I'm writing this
> email to propose an improvement in the campaign, which suffers from
> the hard-to-obtain reviews on pull requests.
>
> The problem: it is hard to get reviews on pull requests, so they may
> be stuck in "need review" state for a long time.
> There are currently 54 such PRs:
> https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aopen+review%3Arequired
>
> Suggested solution: allow logging _reviews_ on the sheet used for
> reporting work performed by partecipating developers:
>
> https://docs.google.com/spreadsheets/d/1F6v4g8Ayb3wIt73rxFKD8h4B95MaTwt-eDit8YvReB0/edit#gid=1300379589
>
> At the moment the sheet contains a line for each task taken by
> each developer. We could add an additional column to report
> the name of the reviewer (to be filled by the PR author) or we
> could have a column to express whether the developer only did
> the review of a PR done by someone else.
>
> In the first case each PR has only one line, in the second case
> each PR will possibly have multiple lines.
>
> What do you think ?
>
> --strk;
>
>   Libre GIS consultant/developer
>   https://strk.kbt.io/services.html
> _______________________________________________
> Qgis-psc mailing list
> Qgis-psc at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/qgis-psc
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-psc/attachments/20220301/f645bf7d/attachment.html>