Re: Add --check option to pgindent - Mailing list pgsql-hackers

From Tristan Partin
Subject Re: Add --check option to pgindent
Date
Msg-id CXRRU5Z7R9XN.3JZTO6G7PTKSU@neon.tech
Whole thread Raw
In response to Re: Add --check option to pgindent  ("Tristan Partin" <tristan@neon.tech>)
Responses Re: Add --check option to pgindent
Re: Add --check option to pgindent
List pgsql-hackers
On Mon Dec 18, 2023 at 10:50 AM CST, Tristan Partin wrote:
> On Mon Dec 18, 2023 at 10:14 AM CST, Jelte Fennema-Nio wrote:
> > On Mon, 18 Dec 2023 at 13:42, Daniel Gustafsson <daniel@yesql.se> wrote:
> > > I think this is pretty much ready to go, the attached v4 squashes the changes
> > > and fixes the man-page which also needed an update.  The referenced Wiki page
> > > will need an edit or two after this goes in, but that's easy enough.
> >
> > One thing I'm wondering: When both --check and --diff are passed,
> > should pgindent still early exit with 2 on the first incorrectly
> > formatted file? Or should it show diffs for all failing files? I'm
> > leaning towards the latter making more sense.
>
> Makes sense. Let me iterate on the squashed version of the patch.

Here is an additional patch which implements the behavior you described.
The first patch is just Daniel's squashed version of my patches.

--
Tristan Partin
Neon (https://neon.tech)

Attachment

pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: index prefetching
Next
From: "Tristan Partin"
Date:
Subject: Re: Add --check option to pgindent