Re: CI: Add task that runs pgindent
| От | Tom Lane |
|---|---|
| Тема | Re: CI: Add task that runs pgindent |
| Дата | |
| Msg-id | 1233967.1762531655@sss.pgh.pa.us обсуждение исходный текст |
| Ответ на | Re: CI: Add task that runs pgindent (Andres Freund <andres@anarazel.de>) |
| Список | pgsql-hackers |
Andres Freund <andres@anarazel.de> writes:
> Why does this need to be a separate task, instead of being folded into the
> CompilerWarnings task or such? Increasing the peak numbers of CPUs cfbot needs
> is unfortunately somewhat annoying, as I've been fighting w/ GCP to increase
> the quotas much higher than what they are right now.
We had that discussion upthread, or maybe it was on Discord: we want
to keep formatting complaints separate from actual compiler warnings.
The latter should be fixed before the patch can be considered ready,
whereas pgindent might get left for the committer to do. An example
where that's actually good practice is where the patch has to reindent
a chunk of existing code; submitting it without that makes review
simpler.
regards, tom lane
В списке pgsql-hackers по дате отправления: