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.
> 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
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" ?
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
marcimat
b_b
à titre d'exemple, mise en place des outils de développement pour un plugin, passe d'autofix et baseline pour phpstan.
dfe8c4506eto0f69a70fdd1 year agoOk. 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.
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
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" ?
Modif manuelle ou par l'outil ? Il y a une erreur dans le reformattage là :
dfe8c4506e/action/editer_url.php (L301)be5ab43101into master 1 year agoReviewers
be5ab43101.