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

From Nazir Bilal Yavuz
Subject Re: run pgindent in CI
Date
Msg-id CAN55FZ1PqQakh702CLDrQxVi+HZW-GZWVrJ6r3ONwak92y5dgg@mail.gmail.com
Whole thread
In response to run pgindent in CI  (Peter Eisentraut <peter@eisentraut.org>)
List pgsql-hackers
Hi,

On Tue, 22 Sept 2026 at 10:11, Peter Eisentraut <peter@eisentraut.org> wrote:
>
> Previously discussed in [0].  People seemed to generally like the idea
> of running pgindent in CI but didn't want the job to fail outright if
> there were violations.  My proposal here is to use a combination of
> GitHub Actions features [1] to create job annotations and step summaries
> to create some kind of warning output.  I have attached an example
> screenshot.
>
> I also added a git whitespace check in the same way.  Additional
> formatting checks like pgperltidy, Python stuff, etc. could be added
> similarly.
>
> Thoughts?

For the 0003, what do you think about just mentioning that 'the
style-check steps in the CompilerWarnings have failed' instead of
printing the diff? That would make sense since we won't actually need
to understand the diff (AFAIK), we just need to run some tools.

--
Regards,
Nazir Bilal Yavuz
Microsoft



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: run pgindent in CI
Next
From: Peter Eisentraut
Date:
Subject: Re: run pgindent in CI