<div dir="ltr"><div>Hi,</div><div><br></div><div>We will discuss this at our PSC meeting on Tue 5.</div><div><br></div><div>Greetings,</div><div>Andreas<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 26 Feb 2024 at 00:21, Nyall Dawson via QGIS-PSC <<a href="mailto:qgis-psc@lists.osgeo.org">qgis-psc@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 PSC,<br>
<br>
I would love for an official call to be made on which of two<br>
conflicting pull request queue management policies should be adopted<br>
by QGIS.<br>
<br>
There are currently two proposals, and the lack of a formal policy is<br>
causing confusion/conflict in how pull requests are managed.<br>
<br>
Policy #1: <a href="https://github.com/qgis/QGIS/pull/56062" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/56062</a><br>
<br>
In short, Sandro proposes that the pull request queue be an open queue<br>
of ALL work happening everywhere, in any state of completeness. Pull<br>
requests are permitted for semi-complete work, and for long-term<br>
(including multi-year) projects which are not yet ready for review or<br>
merge. The justification here is that having this work open in the<br>
queue makes it widely visible and so that other developers are aware<br>
of ongoing work across the community.<br>
<br>
Currently, these pull requests will be auto-closed by stalebot due to<br>
the lack of activity on the ticket. Sandro's proposal is to disable<br>
stalebot handing of draft / WIP pull requests, and effectively to<br>
formalise that the queue is a valid place for work of this nature and<br>
status.<br>
<br>
(@strk please expand here if you feel I haven't summarised your point<br>
of view correctly!)<br>
<br>
Policy #2: <a href="https://github.com/qgis/QGIS/pull/56523" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/56523</a><br>
<br>
In this PR I propose to set a formal policy that draft and WIP pull<br>
requests are NOT suitable for opening against the QGIS repository.<br>
<br>
My justification is that we have a long-standing issue with<br>
maintainability of the pull request queue, and anything which<br>
decreases the signal-to-noise ratio on open tickets is undesirable.<br>
When the queue includes work which is not ready for review, then it<br>
becomes very tricky to work out the actual status of pull requests and<br>
which ones should be focused on during review time. (Effectively right<br>
now we have a situation where any pull request which is pushed on the<br>
2nd page of requests will basically NEVER get reviewed, as there is a<br>
constant stream of ready-for-review work flowing into the first page<br>
and the signal-to-noise ratio of ready-for-review/merge PRs on<br>
subsequent pages is extremely low). I do not believe it is fair for<br>
submissions like <a href="https://github.com/qgis/QGIS/pull/55172" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/55172</a> or<br>
<a href="https://github.com/qgis/QGIS/pull/55293" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/55293</a> where reviews take SUCH a long<br>
time, and it is my belief that by keeping the queue as small as<br>
possible and avoiding WIP/draft work we will increase the likelihood<br>
that PRs like these can be reviewed more quickly in future.<br>
<br>
Please note that there is considerable discussion on<br>
<a href="https://github.com/qgis/QGIS/pull/56062" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/56062</a> already which should be read<br>
when reviewing this decision.<br>
<br>
Can I ask that PSC choose one of these two policies to formally adopt<br>
so that there is no misunderstanding or conflict in future?<br>
<br>
Thanks in advance!<br>
Nyall<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><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>