Обсуждение: Add .editorconfig

There were a couple of recent threads that wanted to add an 
.editorconfig file but never actually ended up doing so.[0][1]  Here is 
a patch.  It is meant to match more or less what's in .dir-locals.el.

I have only tested this with the GitHub view, not with an actual editor.

[0]: 
https://www.postgresql.org/message-id/flat/20180605172252.GA2433%40fetter.org
[1]: 
https://www.postgresql.org/message-id/flat/20180529153617.c72llpzbvt46chr6%40alap3.anarazel.de

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

> On 11 Dec 2019, at 17:00, Peter Eisentraut <peter.eisentraut@2ndquadrant.com> wrote:
>
> There were a couple of recent threads that wanted to add an .editorconfig file but never actually ended up doing
so.[0][1] Here is a patch.  It is meant to match more or less what's in .dir-locals.el. 

+[*.{c,h,l,y,pl,pm}]

What about *.sh?

Not that there is much of Python in the tree, but shouldn't those also be
covered if we do this?

[*.py]
indent_style = space
indent_size = 4

cheers ./daniel

On 2019-12-11 17:09, Daniel Gustafsson wrote:
>> On 11 Dec 2019, at 17:00, Peter Eisentraut <peter.eisentraut@2ndquadrant.com> wrote:
>>
>> There were a couple of recent threads that wanted to add an .editorconfig file but never actually ended up doing
so.[0][1] Here is a patch.  It is meant to match more or less what's in .dir-locals.el.
 
> 
> +[*.{c,h,l,y,pl,pm}]
> 
> What about *.sh?
> 
> Not that there is much of Python in the tree, but shouldn't those also be
> covered if we do this?
> 
> [*.py]
> indent_style = space
> indent_size = 4

Those were not in the Emacs config either.

I'd be inclined to leave the Python stuff alone, in particular.  The 
PEP-8 style appears to be universally accepted as the default, so we 
don't need to specify it here.  We only need to specify what's different 
or where no sensible default exists.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

I have not used .editorconfig that much, but would it makes sense to add 
the below?

[*]
end_of_line = lf

-- 
Andreas Karlsson

On 2019-12-11 18:54, Andreas Karlsson wrote:
> I have not used .editorconfig that much, but would it makes sense to add
> the below?
> 
> [*]
> end_of_line = lf

I think that would best be done in response to an actual need.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

On 2019-12-11 17:00, Peter Eisentraut wrote:
> There were a couple of recent threads that wanted to add an
> .editorconfig file but never actually ended up doing so.[0][1]  Here is
> a patch.  It is meant to match more or less what's in .dir-locals.el.

committed

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services