Re: Add trim_trailing_whitespace to editorconfig file

Jelte Fennema-Nio <postgres@jeltef.nl>

View thread View on postgr.es

From: Jelte Fennema-Nio <postgres@jeltef.nl>

To: Daniel Gustafsson <daniel@yesql.se>

Cc: Melanie Plageman <melanieplageman@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>

Date: 2024-02-15T09:26:34Z

Lists: pgsql-hackers

Attachments

v2-0002-Require-final-newline-in-.po-files.patch (application/octet-stream) patch v2-0002
v2-0003-Bring-editorconfig-in-line-with-gitattributes.patch (application/octet-stream) patch v2-0003
v2-0004-Add-note-about-keeping-.editorconfig-and-.gitattr.patch (application/octet-stream) patch v2-0004
v2-0001-Remove-non-existing-file-from-.gitattributes.patch (application/octet-stream) patch v2-0001

On Wed, 14 Feb 2024 at 23:19, Daniel Gustafsson <daniel@yesql.se> wrote:
> > +1 from me. But when do we want it to be false? That is, why not
> > declare it true for all file types?
>
> Regression test .out files commonly have spaces at the end of the line.  (Not
> to mention the ECPG .c files but they probably really shouldn't have.)

Attached is v2, which now makes the rules between gitattributes and
editorconfig completely identical. As well as improving two minor
things about .gitattributes before doing that.

Commits

Add script to keep .editorconfig in sync with .gitattributes
- 2452e71ff2fe 18.0 landed
Remove non-existing file from .gitattributes
- 1d577f2eaa88 15.7 landed
- 2c7f2eb0c8aa 16.3 landed
- e85732dac0e9 17.0 landed