| Re: [Galette-devel] Vérifications automatisées du code |
[ Thread Index |
Date Index
| More lists.galette.eu/devel Archives
]
- To: devel@xxxxxxxxxxxxxxxx
- Subject: Re: [Galette-devel] Vérifications automatisées du code
- From: Thierry <thierry@xxxxxxxxxxx>
- Date: Fri, 12 Jun 2020 21:58:57 +0200
- Dkim-signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=palijn.info ; s=default; h=Message-Id:In-Reply-To:To:References:Date:Subject:Mime-Version :Content-Transfer-Encoding:Content-Type:From:Sender:Reply-To:Cc:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=kpeKQwRFaDCiijCjQd7lmhaGe032QE9KEa1YBXzcaY0=; b=WkiUuZ2sAfe3agcyLfdctRbrIU n82RQg3p8w77er/1jc0fBEUzBmqPdDg29k6DK2S3ak8XRbrP11/BhR2I6AnKmtTCRvLizgLneneyR t/QXMscGQ2DU5GM6wflFCDlbBDfKI/DFw5a3vNR+WbDkQjhCSYy5N0fYFJvsHgWr9dqkFoncxuePS WLKEb1giCFC9PZDbIISNG2bLzGkS7d2ladjGO+prFps4DDb1SnMmivDRXtz3JDkJV/bWqhZb4z9kE RmEUnqaCJ+LY9QkxfHECHYDlcAZ8qpW0Lr7sZsj6XU3bC3IqxSDkQyqzdVZJGaQJ8OZNRNs3mihxe T2Ri2RnQ==;
Bravo, très pro :)
Effectivement j’ai vu passer les PR verbeuses de Scrutinizer…
mais sur leur contenu c’est toi le plus à même de décider !
Cordialement
Thierry
> Le 12 juin 2020 à 09:53, Johan Cwiklinski <johan@xxxxxxxx> a écrit :
>
> Salut,
>
> Suite aux trop nombreuses remontées de soucis relatifs à des
> oublis/imprécisions dans les espaces de noms, j'ai cherché une solution
> pour éviter à l'avenir.
>
> Déjà, j'ai passé les coding standards de PSR2 à PSR12 - ce n'est pas en
> lien direct, mais ça améliorera la lisibilité du code et ça supporte les
> ajouts plus récents de PHP :)
>
> J'ai donc regardé du côté de Psalm (https://psalm.dev/) et de
> Scrutinizer (scrutinizer-ci.com). Je n'ai pas vraiment cherché plus
> loçin, si vous en avez d'autres... - à voir.
>
> L'un comme l'autre ont pu me détecter des problèmes d'espaces de noms
> (mineurs pour la grande majorité, surtout sur les Exceptions) ; qui ont
> donc été corrigés.
> En revanche, l'un comme l'autre sont très verbeux, et je ne suis pas
> forcément d'accord avec tout ce qu'ils disent... Ça va prendre pas mal
> de temps avant que ce soit bien rôdé.
>
> Psalm se lance avec les tests et autres vérifications... C'est donc sur
> une branche à part pour le moment, j'ai trouvé que c'était assez
> compliqué à configurer, ce ne sera pas trop la joie avec d'autrs
> modifications sur le code.
> Scrutinizer se lance à part, c'est assez utilisé sur github (je le vois
> régulièrement), les options de config sont nombreuses et faciles à trouver.
>
> Le but à terme est que l'un ou l'autre de ces systèmes soit lancé à
> chaque commit pour vérifier les erreurs débiles / récurrentes et venir
> râler quand ça ne va pas ! :)
>
> ++
> --
> Johan
>
--
Galette development - http://galette.eu/
List documentation: https://listengine.tuxfamily.org/lists.galette.eu/devel/