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

From Peter Geoghegan
Subject Re: CI: Add task that runs pgindent
Date
Msg-id CAH2-Wzm674VS63sbv5LBJunH=hJ_n1iBBt0iyUiWwpgDFpp1RA@mail.gmail.com
Whole thread Raw
In response to Re: CI: Add task that runs pgindent  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: CI: Add task that runs pgindent
List pgsql-hackers
On Wed, Oct 22, 2025 at 8:07 AM Daniel Gustafsson <daniel@yesql.se> wrote:
> I like the different shape and symbol, but I would probably keep it green to
> indicate that it's informational rather than actionable.  We don't want a
> flurry of patch re-submissions with only whitespace changes eating CI resources
> when the previous build was successful. Just my €0,02.

I strongly agree that this shouldn't signal to the user that they
really need to fix the problem. I'd like it if this information was
presented in the least obtrusive way possible. Constantly worrying
about a misplaced tab is a waste of time for all concerned.

--
Peter Geoghegan



pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: Remove an unnecessary blank line on the PQisBusy() comments
Next
From: Dean Rasheed
Date:
Subject: Re: Docs and tests for RLS policies applied by command type