Re: CI: Add task that runs pgindent

Поиск
Список
Период
Сортировка
От David G. Johnston
Тема Re: CI: Add task that runs pgindent
Дата
Msg-id CAKFQuwZEnGhGV32A55PvEF9HUhjEfDAoxcp7rDvSNL2KKx8Kdw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: CI: Add task that runs pgindent  (Peter Geoghegan <pg@bowt.ie>)
Ответы Re: CI: Add task that runs pgindent
Список pgsql-hackers
On Thu, Oct 23, 2025, 11:14 Peter Geoghegan <pg@bowt.ie> wrote:
On Wed, Oct 22, 2025 at 8:07 AM Daniel Gustafsson <daniel@yesql.se> wrote:
> I like the different shape and symbol, but I would probably keep it green to
> indicate that it's informational rather than actionable.  We don't want a
> flurry of patch re-submissions with only whitespace changes eating CI resources
> when the previous build was successful. Just my €0,02.

I strongly agree that this shouldn't signal to the user that they
really need to fix the problem. I'd like it if this information was
presented in the least obtrusive way possible. Constantly worrying
about a misplaced tab is a waste of time for all concerned.


My thought is to use the new tags feature to add/remove a "Needs Formatting" tag as the indicator.

David J.







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