Re: run pgindent in CI - Mailing list pgsql-hackers

From Tom Lane
Subject Re: run pgindent in CI
Date
Msg-id 1285147.1790174149@sss.pgh.pa.us
Whole thread
In response to Re: run pgindent in CI  (Peter Eisentraut <peter@eisentraut.org>)
Responses Re: run pgindent in CI
List pgsql-hackers
Peter Eisentraut <peter@eisentraut.org> writes:
> For clarification:  We already have a exclude_file_patterns that 
> excludes many generated files.  It just didn't exclude enough of them. 
> Things were just forgotten.  So this proposal is not introducing a new 
> policy or requirement, it just fixes things so that the implementation 
> of the current apparent policy is complete.

My point is exactly that that list hasn't been maintained, because
there was nothing to remind developers to do so.  So I'm doubtful
that there can meaningfully be claimed to be a policy requiring it.

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.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Salma El-Sayed
Date:
Subject: Re: [GSoC 2026] - B-tree Index Bloat Reduction - Approach & Questions
Next
From: Nathan Bossart
Date:
Subject: Re: Speed up lpad() and rpad() for one-byte padding strings