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 по дате отправления: