Re: CI: Add task that runs pgindent

Поиск
Список
Период
Сортировка
От Jelte Fennema-Nio
Тема Re: CI: Add task that runs pgindent
Дата
Msg-id CAGECzQQSaOp-W6gbNnSdVtNCoOdOiOb+1wO=P8hcC-yvM2Xifg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: CI: Add task that runs pgindent  (Daniel Gustafsson <daniel@yesql.se>)
Ответы Re: CI: Add task that runs pgindent
Список pgsql-hackers
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, and I didn't want to
conflate possible objectsions to that with objections to running
pgindent in CI. Once this CI job is in I wanted to start that
discussion for pgindent (and similarly a discussion on adding a python
formatter/linter).



В списке pgsql-hackers по дате отправления: