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

From Jelte Fennema-Nio
Subject Re: CI: Add task that runs pgindent
Date
Msg-id CAGECzQTotzNaniT-tpz5Eg85tO=B_0wOO2SnOuK0hiXqHwVwMg@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, 22 Oct 2025 at 14:06, 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 understand the concern. So I tried out making the icon green now
(see attached), but it looks a bit weird imo. Unless some others
prefer the green too (or have some other idea for an icon), I'm
inclined to keep it the yellow/orange color it is now. If we actually
do get a bunch of useless re-submissions, we can consider changing it.

Attachment

pgsql-hackers by date:

Previous
From: Shinya Kato
Date:
Subject: Re: Add wal_fpi_bytes_[un]compressed to pg_stat_wal
Next
From: Oleg Tselebrovskiy
Date:
Subject: 001_password.pl fails with --without-readline