<div dir="auto">Thanks for the clarification Nyall, as Andreas wrote we'll discuss it next Tuesday at PSC and will get back to you short after.<div dir="auto"><br></div><div dir="auto">Cheers Marco </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 28 Feb 2024, 00:07 Nyall Dawson, <<a href="mailto:nyall.dawson@gmail.com">nyall.dawson@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 28 Feb 2024 at 08:49, Marco Bernasocchi <<a href="mailto:marco@qgis.org" target="_blank" rel="noreferrer">marco@qgis.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"><div dir="auto">Thanks Nyall, thanks Sandro,<div dir="auto"><br></div><div dir="auto">Just a quick question, is the </div><div dir="auto"> -is:draft </div><div dir="auto">Filter an option at all? I guess it does not reduce the n/r by default but could help?</div><div dir="auto"><br></div><div dir="auto"><a href="https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aopen+-is%3Adraft" target="_blank" rel="noreferrer">https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aopen+-is%3Adraft</a></div></div></blockquote><div><br></div><div>That hides them, yes. But the bigger question remains as to what value having unfinished work in the queue has in the first place. In my opinion if it's not ready for review and merge, it's not actually a "pull REQUEST" and just doesn't belong there. In my proposal there's already mention that short term pull requests can be permitted for the express purpose of performing a CI test run, so after that why is there a need to allow unfinished work to remain in the queue? </div><div><br></div><div>If the need is for others to see and be exposed to unfinished work, then let's look at an extreme example: I have some 100's of branches of work in an unfinished state on my QGIS fork, from proof of concept experiments through to things which I just don't think work well enough to be submitted. Should all of those draft/wip branches be pull requests in the QGIS repo? 🤣 If the justification for allowing draft/wip requests is that others can see what's being/been done, then I guess the same argument would apply to all of this.</div><div><br></div><div>Nyall</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Cheers Marco </div><div dir="auto"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 27 Feb 2024, 23:09 Sandro Santilli via QGIS-PSC, <<a href="mailto:qgis-psc@lists.osgeo.org" target="_blank" rel="noreferrer">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">Thanks Nyall for raising this thread!<br>
<br>
On Mon, Feb 26, 2024 at 09:21:05AM +1000, Nyall Dawson wrote:<br>
<br>
> Policy #1: <a href="https://github.com/qgis/QGIS/pull/56062" rel="noreferrer noreferrer 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>
I would add that allowing long-term PRs also serves the purpose of making<br>
it easy for contributors to check the state of their work against CI as<br>
running tests locally is so hard that I dubt anyone is doing that.<br>
<br>
> Policy #2: <a href="https://github.com/qgis/QGIS/pull/56523" rel="noreferrer noreferrer noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/56523</a><br>
<br>
[...]<br>
<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.<br>
<br>
I think this is a limitation of the software used to get the list<br>
of pull requests needing focus, and should be threated as such.<br>
If github makes it hard to filter on "PR state" we could use a label<br>
to do that.<br>
<br>
--strk;<br>
_______________________________________________<br>
QGIS-PSC mailing list<br>
<a href="mailto:QGIS-PSC@lists.osgeo.org" rel="noreferrer noreferrer" target="_blank">QGIS-PSC@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/qgis-psc" rel="noreferrer noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-psc</a><br>
</blockquote></div>
</blockquote></div></div>
</blockquote></div>