> On 4 Oct 2021, at 12:06, Magnus Hagander <[email protected]> wrote: > > On Sun, Oct 3, 2021 at 3:48 PM Tom Lane <[email protected] > <mailto:[email protected]>> wrote: > Magnus Hagander <[email protected] <mailto:[email protected]>> writes: > > On Sat, Oct 2, 2021 at 7:31 AM Michael Paquier <[email protected] > > <mailto:[email protected]>> wrote: > >> That's the tricky part. It does not really make sense either to keep > >> moving patches that are waiting on author for months. > > > I'm pretty sure this is the original reason for adding this -- to enforce > > that this review happens. > > The CF tool is in no way able to enforce that, though. All it's doing > is making extra work for the CFM. > > I've now deployed this: > https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=commitdiff;h=65073ba7614ba539aff961e59c9eddbbb8d095f9 > > <https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=commitdiff;h=65073ba7614ba539aff961e59c9eddbbb8d095f9> AFAICT this should now allow for WoA patches to be moved to the next CF, but trying that on a patch in the current CF failed with "Invalid existing patch status" in a red topbar. Did I misunderstand what this change was?
-- Daniel Gustafsson https://vmware.com/
