De toute façon, si un jour on décidait de remplacer les tabs par des espaces ou que sais-je encore, on décidera à l'avance de le faire pendant une journée sans commit, sur des dépôts à jour et fusionnés autant que possible.
Donc ce jour-là (pas encore programmé), on enlèvera les ?> en fin de fichier PHP, on appliquera ce qu'on aura décidé pour les indentations, etc. Le 6 septembre 2013 11:12, Nicolas <[email protected]> a écrit : > Le 6 septembre 2013 10:58, Bruno <[email protected]> a écrit : > > > > > > > Tout à fait. On pourrait même mettre un hook dans mercurial pour > rejeter > > > > > automatiquement les fichiers qui ne respecteraient pas les conventions. > > > > > > > +1, mais il faudrait déjà qu'on ait "le cul propre" sur tous les fichiers > > > > > Je parlais des nouveaux commit mais je suis d'accord il faudrait commencer > par faire une passe plus ou moins automatique pour avoir un tout cohérent. > > > > > > Globalement les conventions se retrouvent d'un projet à l'autre mis à > > part > > > les tabulations. Les tabulations caymal ! > > > > > Je ne veux surtout pas tout remettre en question, ni forcer à utiliser > PSR2 > > privé de PSR0 et PSR1 (sinon ça fait ça : http://xkcd.com/927/ ), mais > > juste qu'on soit tous d'accord sur ce qu'on va mettre en place partout > dans > > le code. Tabulations ou pas, j'avoue que je n'ai pas d'avis dessus, c'est > > juste une histoire de configuration d'éditeur après tout :) > > > > > Je pense que les conventions actuelles sont correctes. > > > > > > > > > > Je suis bien conscient que cela risque d'impacter *tous* les fichiers > > de > > > > CB/DC, mais avec un peu de chance, ça se fait en une seule fois, avec > > la > > > > bonne moulinette :) > > > > > > > > > > On pourrait commencer par virer les ?> de fin de fichier php et ajouter > > > systématiquement une ligne vide en fin de fichier pour tous les > fichiers. > > > > > > > tout à fait, oui :) > > > > Bernard avait tenter de faire une PR sur le sujet. Je ne sais pas ce qu'il > en est. > > > > -- > > Dev mailing list - [email protected] - > > http://ml.dotclear.org/listinfo/dev > > > -- > Dev mailing list - [email protected] - > http://ml.dotclear.org/listinfo/dev > -- Franck -- Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev
