feat(dev-tools): Mise en place phpcs+phpstan #4807

Merged
JamesRezo merged 2 commits from qa-tools into master 1 year ago
JamesRezo commented 1 year ago
Owner

à titre d'exemple, mise en place des outils de développement pour un plugin, passe d'autofix et baseline pour phpstan.

à titre d'exemple, mise en place des outils de développement pour un plugin, passe d'autofix et baseline pour phpstan.
JamesRezo force-pushed qa-tools from dfe8c4506e to 0f69a70fdd 1 year ago
marcimat approved these changes 1 year ago
marcimat left a comment
Owner

Ok. Du beau code formatté.

Cependant ce fichier .neon me questionne avec tout ce contenu ignoré (les fonctions utilisées de SPIP non présentes dans le plugin). Enfin bon.

Ok. Du beau code formatté. Cependant ce fichier .neon me questionne avec tout ce contenu ignoré (les fonctions utilisées de SPIP non présentes dans le plugin). Enfin bon.
b_b approved these changes 1 year ago
Poster
Owner

Cependant ce fichier .neon me questionne avec tout ce contenu ignoré (les fonctions utilisées de SPIP non présentes dans le plugin). Enfin bon.

Oui, c'est légitime. Avant de merger cette PR et de généraliser aux autres plugins-dist, je vais apporter 2/3 reflexions du discuter là-dessus : https://discuter.spip.net/t/analyse-statique-du-code/155403/8

> Cependant ce fichier .neon me questionne avec tout ce contenu ignoré (les fonctions utilisées de SPIP non présentes dans le plugin). Enfin bon. Oui, c'est légitime. Avant de merger cette PR et de généraliser aux autres plugins-dist, je vais apporter 2/3 reflexions du discuter là-dessus : https://discuter.spip.net/t/analyse-statique-du-code/155403/8
b_b commented 1 year ago
Owner

Je viens d'approuver toute la série :) Vu que le process est un peu long pour ajouter une review à cause du temps d'affichage de la page du diff, est-ce qu'on peut se mettre d'accord ici pour la complète et merger la complète "à la chaîne" ?

Je viens d'approuver toute la série :) Vu que le process est un peu long pour ajouter une review à cause du temps d'affichage de la page du diff, est-ce qu'on peut se mettre d'accord ici pour la complète et merger la complète "à la chaîne" ?
JLuc commented 1 year ago

Modif manuelle ou par l'outil ? Il y a une erreur dans le reformattage là : dfe8c4506e/action/editer_url.php (L301)

Modif manuelle ou par l'outil ? Il y a une erreur dans le reformattage là : https://git.spip.net/spip/urls_etendues/src/commit/dfe8c4506e49bffac62419bce0e348f59fa72799/action/editer_url.php#L301
JamesRezo added 1 commit 1 year ago
JamesRezo merged commit be5ab43101 into master 1 year ago
JamesRezo deleted branch qa-tools 1 year ago

Reviewers

marcimat approved these changes 1 year ago
b_b approved these changes 1 year ago
The pull request has been merged as be5ab43101.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.