-
Notifications
You must be signed in to change notification settings - Fork 320
Migrer PHPStan vers une dépendance Composer #3110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: alpha
Are you sure you want to change the base?
Conversation
323ed18
to
cbea5e6
Compare
De mémoire : Mais si le faire pour les 2 versions why not. Après je suis pas certain que ca fasse une différence sur le résultat. |
Avec composer on a la version de phpstan compatible php7.4. Et comme ça on valide aussi en php8.2, si jamais quelqu'un propose un PR qui n'est pas compatible avec les 2 versions de PHP on le détectera :) |
Ok, faudrait tester dans ce cas : rajouter volontairement une modification qui soit une erreur dans une version, pas dans une autre. Le premier exemple qui me vient à l'esprit c'est l'opérateur |
aa05794
to
adf20cf
Compare
adf20cf
to
b2bf4cf
Compare
@pifou25 tu pourras faire un update de phpstan/phpstan stp, y a un fix qui impacte le dépôt qui est passé récement. |
à quoi bon ? la PR a déjà 3 mois j'essaye laborieusement de faire un rebase de temps en temps... sur la PR c'est phpstan:^2.1 et la dernière dispo à l'instant où j'écris ces lignes c'est 2.1.30 donc, c'est bon non ? |
Description
Fixes #3106
Mais aussi, l'action était ambigue avec la double utilisation de php7.4 et 8.2 dans le même job. Alors j'ai séparé en mode matrice pour jouer 2 fois le job, une fois en php7.4 et l'autre en php8.2

Pour la partie baseline, j'ai appliqué le même pattern matriciel, mais je ne sais pas si c'est opportun ?
Suggested changelog entry
Related issues/external references
Fixes #3106
Types of changes
PR checklist