Bonjour, Le 6 septembre 2013 10:28, Bruno <[email protected]> a écrit :
> Ca m'a fait tout drôle d'activer les règles par défaut de phpcs sur > quelques pages de l'admin... > Je ne connais pas et n'utilise donc pas phpcs. J'utilise https://github.com/fabpot/PHP-CS-Fixer Mais dans le principe c'est la même chose. > Histoire de faire un peu de QA, peut-on se mettre d'accord sur les > conventions de codage ? Point encore question de parler de PSR-0/1/2, au > moins se mettre d'accord sur la mise en forme des fichiers PHP/js > (indentation, espaces, ...) histoire de se faire un fichier de règles PHPCS > qui permette d'avoir un code cohérent partout. > Tout à fait. On pourrait même mettre un hook dans mercurial pour rejeter automatiquement les fichiers qui ne respecteraient pas les conventions. > > Ce serait l'occasion de discuter du règles édictées là : > > http://fr.dotclear.org/documentation/2.0/resources/coding-standards#code-php > ... et de les confronter avec les meilleurs pratiques des projets PHP... > Globalement les conventions se retrouvent d'un projet à l'autre mis à part les tabulations. Les tabulations caymal ! > > 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. Nicolas -- Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev
