Re: CI: Add task that runs pgindent
Daniel Gustafsson <daniel@yesql.se>
From: Daniel Gustafsson <daniel@yesql.se>
To: Jelte Fennema-Nio <me@jeltef.nl>
Cc: Nazir Bilal Yavuz <byavuz81@gmail.com>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-10-21T15:04:50Z
Lists: pgsql-hackers
> On 21 Oct 2025, at 16:31, Jelte Fennema-Nio <me@jeltef.nl> wrote: > > On Tue, 21 Oct 2025 at 15:27, Daniel Gustafsson <daniel@yesql.se> wrote: >> >>> On 21 Oct 2025, at 15:22, Nazir Bilal Yavuz <byavuz81@gmail.com> wrote: >> >>> Do you think that this task should format perl files as well? >> >> +1, if we do this we should run pgperltidy as well. > > I definitely agree that we should run that too. But currently the > master branch isn't correctly formatted according to pgperltidy (at > least with the pgperltidy on my machine I get a bunch of changes). So > I guess koel doesn't complain about that yet, Indeed there is, I admittedly was under the impression that koel ran pgperltidy but clearly I was wrong. -- Daniel Gustafsson