Re: Add "format" target to make and ninja to run pgindent and pgperltidy - Mailing list pgsql-hackers

From Jelte Fennema-Nio
Subject Re: Add "format" target to make and ninja to run pgindent and pgperltidy
Date
Msg-id DGTVWJWZB0C9.CKSLBGGSXN4F@jeltef.nl
Whole thread Raw
In response to Re: Add "format" target to make and ninja to run pgindent and pgperltidy  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-hackers
On Wed Dec 31, 2025 at 4:48 PM CET, Andrew Dunstan wrote:
> I'd kinda like to unify these universes, though. We could import the
> pgperltidy logic into pgindent but disable it unless some flag were
> given and the correct version of perltidy were present.

Attached is an initial patchset that does that, as well as improving
pgindent in various other ways. This has removed the make/meson
integration and instead made pgindent smarter at finding pg_bsd_indent
and perltidy. I believe that with all of these QoL improvements we could
enable perltidy in pgindent by default (after doing a full indent run).

I want to call pretty much all of this code was written by Claude Code
(i.e. an AI). I plan to do another review pass over this code soonish.
Especially the later commits, which I haven't checked in detail yet.
But for now I at least wanted to share the direction of what I've been
working on, because I saw that Peter marked himself as reviewer on the
commitfest app and I don't think it makes sense for him to look at the
previous patchset.

Attachment

pgsql-hackers by date:

Previous
From: Alexander Lakhin
Date:
Subject: Re: BUG: Former primary node might stuck when started as a standby
Next
From: shveta malik
Date:
Subject: Re: Skipping schema changes in publication