| From: | Jelte Fennema-Nio <me(at)jeltef(dot)nl> |
|---|---|
| To: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: CI: Add task that runs pgindent |
| Date: | 2025-10-21 14:36:51 |
| Message-ID: | CAGECzQQ+S3Onw7zBMGCNCviqxELw0VMNQWh132c1ss9OjLztgw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, 21 Oct 2025 at 15:22, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)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.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2025-10-21 14:38:41 | Re: another autovacuum scheduling thread |
| Previous Message | Jelte Fennema-Nio | 2025-10-21 14:31:00 | Re: CI: Add task that runs pgindent |