Mise en place de PHPStan dans la CI

[Mopolo]

Cette PR met en place PHPStan dans la CI et en local.

Le projet utilisant une ancienne version de PHP (et de certaines dépendances), l'exécution de PHPStan n'est pas aisée.

En local

Pour exécuter PHPStan en local il faut utiliser Docker car l'outil ne supporte pas PHP inférieur à 7.2 :

make phpstan

# La commande derrière est :
docker run -v $(shell pwd):/app --rm ghcr.io/phpstan/phpstan

Note

PHPStan est configuré pour considérer que le code analysé est en 7.4, vu que c'est la version vers laquelle tend le projet en ce moment, cf #1330
Et aussi parceque PHPStan n'analyse pas de code inférieur à la version 7.0.

Dans la CI

Le job utilise PHP 8.2, ce qui n'a pas d'incidence sur l'analyse vu qu'elle est configurée en 7.4 et ça fait une CI de moins à mettre à jour plus tard.

Le package setasign/fpdf a été mis à jour en 1.6 pour corriger un bug totalement bloquant pour PHPStan
=> Corrigé dans la PR #1371

Le niveau d'analyse

Pour le moment PHPStan est configuré pour analyser au niveau le plus bas (le niveau 1). Dès le second niveau on passe à 179 erreurs et je préfère avoir l'outil dans la CI avant de continuer.

[vinceAmstoutz]

Beau travail Nathan ! Juste pour info le level 1 de PHPStan est le niveau 2 pas le 1 😉PHPStan Rules levels

[Mopolo]

Beau travail Nathan ! Juste pour info le level 1 de PHPStan est le niveau 2 pas le 1 😉PHPStan Rules levels

Ah oui bien vu !

[agallou]

@Mopolo la PR #1371 a aussi modifié la dépendance envers FPDF, tu pourrais rebaser pour résoudre les conflits avant qu'on merge la PR ?