J’ai entrepris de mettre à jour les outils que j’utilise pour coder, en particulier php-cs-fixer qui permet de formater le code PHP en fonction d’un ensemble de règles communément partagées (PSR entre autres).
Sauf qu’au passage je suis passé de la version 2 à la version 3 de l’outil et ça n’est pas anodin, le fichier de configuration ayant changé de nom et de contenu (peu ou prou) — j’ai d’ailleurs poussé un fichier de configuration à utiliser avec cette nouvelle version —, certaines règles ont disparu, etc.
Conséquence l’aspect du code PHP va changer un peu, mais uniquement la forme, alors je ferai une passe complète sur le code de Clearbricks et de Doclear après la prochaine version 2.20 ; en attendant la fin de la semaine prochaine, date de sortie prévue, il n’est toutefois pas impossible que certains fichiers soient reformatés à l’occasion d’une correction ou d’un léger changement.
Bref, un gros commit à venir :-)
1 De Franck -
Finalement, j’ai fait une passe sur Clearbricks (mais ça attendra après la 2.20 pour que ce soit intégré à Dotclear), et il n’y a pas tant de changements de forme que ça…
Bonne nouvelle !
2 De JcDenis -
Etant frileux envers composer, à reculons, j’ai testé pour la première fois cet outil, ça fait du bruit :p
Mais efficace. Je vais faire une passe sur les plugins que je maintient en ce moment en utilisant la même config que Dotclear.
3 De JcDenis -
Par contre une des règles actuelles est de ne pas laisser de ligne vide en fin de fichier, tu ne l’appliques pas pour Dotclear ?
4 De Franck -
Ah oui effectivement, peut-être parce qu’un autre des plugins que j’utilise (sous Sublime Text), voire l’éditeur lui-même qui est réglé comme ça…
Et effectivement dans les settings de ST j’ai un :
"ensure_newline_at_eof_on_save": true,;-)
5 De Franck -
Je vais voir s’il faut modifier les fichiers de configuration de php-cs-fixer car après lecture attentive de la « norme » PSR-2 : “All PHP files MUST end with a single blank line.”, ce qui est assez clair :-)
6 De JcDenis -
PSR-2 c’est has-been, vive PSR-12 ! 😂
Et dans la 12, histoire de changer tous les 6 mois :
7 De JcDenis -
Bon ils se contredisent, un peu plus loin ils déclarent l’inverse…
8 De Franck -
Tiens il faudrait que je teste le preset @PSR12 et qu’au passage je fasse un peu le ménage dans toutes les règles additionnelles…