Re: Adding a clang-format file

From: Aleksander Alekseev <aleksander(at)timescale(dot)com>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Cc: Peter Geoghegan <pg(at)bowt(dot)ie>
Subject: Re: Adding a clang-format file
Date: 2022-09-27 12:34:48
Message-ID: CAJ7c6TNeTUPrjfNbfZQ7hREo5FEZDLavm68MSGvf-_X2aZgPPA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Peter,

> I really don't see any real problem with making something available,
> without changing any official project guidelines. It's commonplace to
> provide a clang-format file these days.

Personally I don't have anything against the idea. TimescaleDB uses
clang-format to mimic pgindent and it works quite well. One problem
worth mentioning though is that the clang-format file is dependent on
the particular version of clang-format. TSDB requires version 7 or 8
and the reason why the project can't easily switch to a newer version
is that the format has changed. So perhaps a directory would be more
appropriate than a single file.

--
Best regards,
Aleksander Alekseev

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2022-09-27 12:42:49 Re: A doubt about a newly added errdetail
Previous Message houzj.fnst@fujitsu.com 2022-09-27 12:31:15 RE: Perform streaming logical transactions by background workers and parallel apply