Re: CI: Add task that runs pgindent - Mailing list pgsql-hackers

From Jelte Fennema-Nio
Subject Re: CI: Add task that runs pgindent
Date
Msg-id CAGECzQQ+S3Onw7zBMGCNCviqxELw0VMNQWh132c1ss9OjLztgw@mail.gmail.com
Whole thread Raw
In response to Re: CI: Add task that runs pgindent  (Nazir Bilal Yavuz <byavuz81@gmail.com>)
List pgsql-hackers
On Tue, 21 Oct 2025 at 15:22, Nazir Bilal Yavuz <byavuz81@gmail.com> wrote:
> What do you think about moving this inside of another CI task instead
> of creating a new one? I think that creating a new VM and cloning
> Postgres into it would be unnecessary but I could not decide which
> task would be a good choice for this.

I think the main benefit of keeping it separate is that it makes it
clear to both reviewers and authors that only the formatting is
incorrect, and no functionality is broken. The task is pretty quick
(~40 seconds) so I wouldn't be too worried about the resources it
uses.



pgsql-hackers by date:

Previous
From: Jelte Fennema-Nio
Date:
Subject: Re: CI: Add task that runs pgindent
Next
From: Nathan Bossart
Date:
Subject: Re: another autovacuum scheduling thread