[QGIS-Developer] Regular paid positions for PR reviews

[Germán Carrillo]

Hi All,

Thanks Alessandro for raising this.

First of all, thanks a lot to all PR reviewers for their outstanding work.

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.

Hopefully, that will reduce the number of PRs that go to 'Stale' status,
which might be a bit discouraging sometimes.
This is a topic we discussed with other QGIS contributors in the latest *QGIS
contributor meeting* in Vienna, and they had the same feeling.


Regards,

Germán


El vie, 8 dic 2023 a las 3:35, Alessandro Pasotti via QGIS-Developer (<
qgis-developer at lists.osgeo.org>) escribió:

> On Fri, Dec 8, 2023 at 9:22 AM Andreas Neumann <andreas at qgis.org> wrote:
> >
> > Dear Alex,
> >
> > I support this idea. Code reviews are important.
> >
> > 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.
> >
> > Are we missing anything?
>
> I think the idea was to go from voluntary random PR review
> contributions to a regular commitment we can count upon.
>
> >
> > 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.
>
> That's exactly the point, some of us think it would be better the other
> way.
>
> >
> > We can formalize this a bit better if you want. I am open for
> suggestions how to improve the situation.
> >
> > 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.
>
> No, I would say we keep it separate, I agree that during the
> bug-fixing time the participants can spend their time doing both
> bugfixing and PR reviews provided that the reviews are oriented to bug
> fixing/chores and not to new features.
>
> >
> > It is a really good timing to discuss this now - because we want to
> bring the 2024 budget to vote next week.
>
> I look forward to other developers' opinions.
>
> Kind regards.
>
> >
> > Greetings,
> > Andreas
> >
> > On Fri, 8 Dec 2023 at 08:57, Alessandro Pasotti via QGIS-Developer <
> qgis-developer at lists.osgeo.org> wrote:
> >>
> >> Hi,
> >>
> >> Recently I have had the opportunity to discuss with some core QGIS
> >> developers and we agreed that it would be a good idea if the QGIS
> >> organization could fund a few weekly or monthly hours for PR reviews.
> >>
> >> Here is my quick and initial proposal of requisites for the candidates:
> >>
> >> - must be an active well known and long time QGIS C++ core developer
> >> (for example from the top last 5 years committers [1])
> >> - ideally we'd need more than a single developer: there is not a
> >> single developer who is able to review all different areas of the code
> >> base
> >> - available for regular work (fixed number of weekly or monthly hours,
> >> to be determined)
> >>
> >> Note: the above list is just my quick ideas, feel free to suggest
> >> anything you think best.
> >>
> >>  I would like this topic to be added to the next PSC meeting agenda.
> >>
> >> Kind regards.
> >>
> >>
> >> [1]
> https://github.com/qgis/QGIS/graphs/contributors?from=2018-12-08&to=2023-12-08&type=c
> >>
> >> --
> >> Alessandro Pasotti
> >> QCooperative:  www.qcooperative.net
> >> ItOpen:   www.itopen.it
> >> _______________________________________________
> >> QGIS-Developer mailing list
> >> QGIS-Developer at lists.osgeo.org
> >> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> >> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> >
> >
> >
> > --
> >
> > --
> > Andreas Neumann
> > QGIS.ORG board member (treasurer)
>
>
>
> --
> Alessandro Pasotti
> QCooperative:  www.qcooperative.net
> ItOpen:   www.itopen.it
> _______________________________________________
> QGIS-Developer mailing list
> QGIS-Developer at lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>


-- 
-----------
   |\__
(:>__)(
   |/
Soluciones Geoinformáticas Libres
http://geotux.tuxfamily.org/
https://twitter.com/GeoTux2 <http://twitter.com/GeoTux2>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-developer/attachments/20231208/3bc992b9/attachment.htm>