Re: Add --check option to pgindent

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Tristan Partin <tristan(at)neon(dot)tech>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Add --check option to pgindent
Date: 2023-12-12 00:21:36
Message-ID: C2F4CA6C-0334-4BD1-B614-300F3B481A05@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 12 Dec 2023, at 01:09, Tristan Partin <tristan(at)neon(dot)tech> wrote:
>
> Not sold on the name, but --check is a combination of --silent-diff and --show-diff. I envision --check mostly being used in CI environments. I recently came across a situation where this behavior would have been useful. Without --check, you're left to capture the output of --show-diff and exit 2 if the output isn't empty by yourself.

I haven’t studied the patch but I can see that being useful.

./daniel

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Junwang Zhao 2023-12-12 02:09:03 Re: Make COPY format extendable: Extract COPY TO format implementations
Previous Message Tristan Partin 2023-12-12 00:09:21 Add --check option to pgindent