[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Dolibarr-dev] Indent tabs or spaces?

From: Doursenaud , Raphaël
Subject: Re: [Dolibarr-dev] Indent tabs or spaces?
Date: Tue, 6 Jan 2015 09:19:08 +0100

Lenient rules are softly enforced…
Tried to push embracing and enforcing the full PSR for years.
I'm kind of tired at that.

2015-01-06 2:12 GMT+01:00 Marcos García <address@hidden>:
So, are we following PSR-2 or not?

I am seeing Pull Request accepted that are not following it and I'm quite surprised right now.

Marcos García

2012-08-04 17:39 GMT+02:00 Laurent Destailleur (eldy) <address@hidden>:
Well, there is so many way of working, so this is what i suggest :

* We user PSR2 :

* Except for indentation that must be set to:  Tabs = 4 spaces and we don't replace tabs with spaces but we keep the tab (PSR2 says we mut replace space by tabs). Reason is:
This is the default option for Eclipse (Eclipse does not replace with space by default), so most developer will work like this. And above all, if you replace tabs with spaces, there is no way to use ALT-TAB to reduce nb of tabs: when we want to reduce a part of indented code of one level, we must remove manually spaces on each lines. Very boring.

So I update the wiki to add this on page to explain all options to match this rules :

Le 03/08/2012 18:27, Régis Houssin a écrit :
c'est dans PSR-2

Le 03/08/12 18:17, Doursenaud, Raphaël a écrit :
Wow !

What about the PSR-1 standard you prompted to use some time ago ?
It actully says on line 2 « code MUST use 4 spaces for indenting, not tabs »

Laurent's reply in the thread was also very wise as this doesn't matter as long as everyone uses 4 as their tab setting in their favorite editor.

I prefer spaces thou, because it doesn't  look ugly whatever the settings are.


2012/8/3 Régis Houssin <address@hidden>

in eclipse, a tab corresponding to 4 spaces by default

Le 03/08/12 14:35, Marcos García a écrit :

One question, what do you officially use to indent in Dolibarr, tabs or spaces?

Marcos García

Raphaël Doursenaud
Directeur technique (CTO)
Expert certifié en déploiement Google Apps
+33 (0)5 35 53 97 13 - +33 (0)6 68 48 20 10
Technopole Hélioparc
2 avenue du Président Pierre Angot
64053 PAU CEDEX 9
SARL au capital de 7 500 € - R.C.S. PAU 528 995 921

reply via email to

[Prev in Thread] Current Thread [Next in Thread]