Re: run pgindent in CI

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: run pgindent in CI
Date: 2026-09-24 15:15:28
Message-ID: a2daccef-e3e4-4216-a5bd-806ffc1e77e6@eisentraut.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 23.09.26 16:35, Tom Lane wrote:
> If we want to change that, then making CI bitch about non-ignored
> files would be a good step, but it's incomplete unless the buildfarm
> implementation enforces indent cleanliness the same way. IOW, what
> I'm complaining about here is that what you want to do doesn't match
> what koel does. They need to match.

I have launched two new threads to prepare for this:

pgindent to ignore build directories
(https://www.postgresql.org/message-id/fc77b35a-34c4-4229-82f1-8cf0ce67f276%40eisentraut.org)

convert CI CompilerWarnings job to meson
(https://www.postgresql.org/message-id/e52e2c29-4834-4c64-a200-eddcba38fb4d%40eisentraut.org)

When those are done, it will be possible to reproduce the koel
environment on CI, and we can come back to this.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message shihao zhong 2026-09-24 15:22:50 Re: [Patch] New pg_stat_tablespace view
Previous Message Merlin Moncure 2026-09-24 15:15:22 Re: Up to 50x degradation in dblink performance when receiving notice traffic 19 vs 18