Add .editorconfig

Started by Peter Eisentrautabout 6 years ago6 messages
#1Peter Eisentraut
peter.eisentraut@2ndquadrant.com
1 attachment(s)

There were a couple of recent threads that wanted to add an
.editorconfig file but never actually ended up doing so.[0]/messages/by-id/20180605172252.GA2433@fetter.org[1]/messages/by-id/20180529153617.c72llpzbvt46chr6@alap3.anarazel.de 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]: /messages/by-id/20180605172252.GA2433@fetter.org
/messages/by-id/20180605172252.GA2433@fetter.org
[1]: /messages/by-id/20180529153617.c72llpzbvt46chr6@alap3.anarazel.de
/messages/by-id/20180529153617.c72llpzbvt46chr6@alap3.anarazel.de

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

Attachments:

0001-Add-.editorconfig.patchtext/plain; charset=UTF-8; name=0001-Add-.editorconfig.patch; x-mac-creator=0; x-mac-type=0Download
From fcf66f8a2e29818b55b0f2c4754bd8bef15bc546 Mon Sep 17 00:00:00 2001
From: Peter Eisentraut <peter@eisentraut.org>
Date: Wed, 11 Dec 2019 16:53:48 +0100
Subject: [PATCH] Add .editorconfig

---
 .editorconfig | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
 create mode 100644 .editorconfig

diff --git a/.editorconfig b/.editorconfig
new file mode 100644
index 0000000000..d69a3d1dc4
--- /dev/null
+++ b/.editorconfig
@@ -0,0 +1,14 @@
+root = true
+
+[*.{c,h,l,y,pl,pm}]
+indent_style = tab
+indent_size = tab
+tab_width = 4
+
+[*.{sgml,xml}]
+indent_style = space
+indent_size = 1
+
+[*.xsl]
+indent_style = space
+indent_size = 2
-- 
2.24.0

#2Daniel Gustafsson
daniel@yesql.se
In reply to: Peter Eisentraut (#1)
Re: Add .editorconfig

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

#3Peter Eisentraut
peter.eisentraut@2ndquadrant.com
In reply to: Daniel Gustafsson (#2)
Re: Add .editorconfig

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

#4Andreas Karlsson
andreas@proxel.se
In reply to: Peter Eisentraut (#1)
Re: Add .editorconfig

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

[*]
end_of_line = lf

--
Andreas Karlsson

#5Peter Eisentraut
peter.eisentraut@2ndquadrant.com
In reply to: Andreas Karlsson (#4)
Re: Add .editorconfig

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

#6Peter Eisentraut
peter.eisentraut@2ndquadrant.com
In reply to: Peter Eisentraut (#1)
Re: Add .editorconfig

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