Repository navigation
#80 [CI] feat: PHPStan, lint PHP et parité des langues - #81
Merged
Merged
Conversation
Meme outillage que les dix autres modules du parc : php -l sur tout l'arbre, parite fr_FR / en_US, et PHPStan niveau 8 avec baseline sur les pull requests. Cinq dossiers du coeur manquaient au gabarit et ont ete ajoutes au scanDirectories et au sparse-checkout ensemble : adherents, compta/bank, compta/paiement, stripe et variants. La bibliotheque Stripe elle-meme est scannee aussi : le trigger appelle \Stripe\Stripe::setApiKey() au paiement. La parite a trouve trois cles mortes en en_US, restes du gabarit modulebuilder (« Consult myobject of WPshop ») : le code utilise les variantes suffixees du nom du module. ReadRight, CreateRight et DeleteRight sont des cles generiques, donc leur presence en en_US sans equivalent fr_FR faisait basculer en anglais tous les libelles de la requete, dans tous les modules. Le trigger redeclare $version avec son type, sans quoi la baseline figerait le numero et la prochaine release casserait la CI. Baseline initiale : 152 entrees. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #80
Même outillage que les dix autres modules du parc :
php -lsur tout l'arbre, parité fr_FR / en_US, et PHPStan niveau 8 avec baseline sur les pull requests.Ce que le gabarit ne couvrait pas, dérivé des
require_oncedu module :adherents,compta/bank,compta/paiement,stripe,variantsincludes/stripe/stripe-php/lib\Stripe\Stripe::setApiKey()au paiementscanDirectorieset le sparse-checkout du workflow sont tenus en phase.La parité a trouvé trois clés mortes en en_US —
ReadRight,CreateRight,DeleteRight, restes du gabarit modulebuilder (« Consult myobject of WPshop »). Le code utilise les variantes suffixées du nom du module, qui existent bien des deux côtés. Ces trois-là sont des clés génériques : leur présence en en_US sans équivalent fr_FR faisait basculer en anglais tous les libellés de la requête, dans tous les modules, pas seulement ici.Le trigger redéclare
$versionavec son type. Sans cela, PHPStan la refuse — le PHPDoc du cœur la type comme une énumération — et la baseline fige le numéro, ce qui casse la CI à chaque release. Le cas s'est produit deux fois sur d'autres modules la semaine dernière.Vérifié en local sur un banc monté avec le cœur Dolibarr et Saturne à jour :
Baseline initiale : 152 entrées.
🤖 Generated with Claude Code