[Résolu] Utiliser la configuration de phpstan dans avec grumphp

Alyve · 04-05-2020 23:14:31

Bonsoir,

Petite question à laquelle je ne trouve réponse. J'ai installé deux librairies sur un projet: GrumPHP et Phpstan pour améliorer la qualité de mon code.
Les deux s'exécutent parfaitement un par un:

Phpstan:

Grumphp:

Le problème c'est quand j'essaie de lancer Phpstan dans le fichier de configuration de Grumphp, celui-ci ne prends pas en compte le fichier de configuration de Phpstan.

grumpphp.yml:

parameters:

    tasks:

        phpstan:

            configuration: phpstan.neon

phpstan.neon:

includes:

    - ./vendor/nunomaduro/larastan/extension.neon

parameters:

    paths:

        - app

    # The level 8 is the highest level

    level: 0

    ignoreErrors:

        - '#Unsafe usage of new static#'

    excludes_analyse:

        - ./*/*/FileToBeExcluded.php

    checkMissingIterableValueType: false

Du coup ma question c'est: comment utiliser phpstan.neon dans grumphp.yml pour qu'il utilise correctement la configuration ?

Merci d'avance ^^'

Caly

Dernière modification par Alyve (06-05-2020 13:00:13)

vv222 · 05-05-2020 15:34:40

Est-ce que l’intégralité de ton fichier de configuration est ignoré, ou seulement certains réglages comme le niveau de sévérité ?

Ici par exemple je vois que je règle le niveau de l’analyse dans la configuration de GrumPHP plutôt que celle de phpstan :

grumphp.yml

parameters:

    git_dir: .

    bin_dir: vendor/bin

    stop_on_failure: true

    tasks:

        (…)

        phpstan:

            metadata:

                priority: 40

            level: 8

phpstan.neon

parameters:

    autoload_directories:

         - database/migrations

    excludes_analyse:

         - 'app/Exceptions'

        - 'app/Http/Controllers/Auth'

        - 'app/Http/Kernel.php'

        - 'app/Http/Middleware'

        - 'app/Models/User.php'

        - 'app/Providers'

        - 'database/factories/UserFactory.php'

        - 'database/migrations'

        - 'server.php'

Alyve · 05-05-2020 15:39:13

Merci pour ta réponse.

Il semblerait que Grum ne lit simplement pas le fichier de Phpstan.

Dernière modification par Alyve (05-05-2020 15:46:30)

vv222 · 05-05-2020 15:45:49

Je t’invite à tester ce grumphp.yml minimal :

parameters:

    tasks:

        phpstan:

            level: 0

Avec bien sûr ton fichier de configuration nommé phpstan.neon à côté.

Est-ce que tu observes une amélioration, ou est-ce que la configuration de phpstan est toujours ignorée ?

Alyve · 05-05-2020 15:48:05

Elle est toujours ignorée, y compris le niveau. En lançant phpstan, pas d'erreur mais avec Grum oui.
Voici la totalité de ce que j'ai actuellement: https://debian-facile.org/paste-A8182215DF

Dernière modification par Alyve (05-05-2020 15:57:16)

vv222 · 05-05-2020 18:49:32

Et pourtant ton fichier de configuration est explicitement indiqué comme pris en compte :

Note: Using configuration file /home/alysson/PhpstormProjects/myproject/phpstan.neon

D’ailleurs je pense que s’il ne l’était pas il y aurait bien plus d’erreurs rapportées sur un Laravel tout frais.

Tu as testé en appelant GrumPHP sans fichier phpstan.neon pour comparer ?

Alyve · 05-05-2020 18:57:03

Il y a bien une différence avec et sans.

Avec:

 [ERROR] Found 10 errors                                                        

Note: Using configuration file /home/alysson/PhpstormProjects/myproject/phpstan.neon.

To skip commit checks, add -n or --no-verify flag to commit command

Sans:

 [ERROR] Found 53 errors

To skip commit checks, add -n or --no-verify flag to commit command

Mais ça me semble étrange, normalement en niveau 0 il ne devrait y avoir aucune erreur.

Edit @22h30
Je viens d'essayer de commit mes modifications en laissant une erreur au niveau 0, et Grumphp l'a prise en compte, elle et uniquement elle. Du coup, ça fonctionne normalement malgré une différence entre l'exécutable dans /vendor/bin et celui du commit… bref, moi pas comprendre mais du coup le soucis est résolu ^^

Dernière modification par Alyve (05-05-2020 21:37:55)

Debian-facile

#1 04-05-2020 23:14:31

[Résolu] Utiliser la configuration de phpstan dans avec grumphp

#2 05-05-2020 15:34:40

Re : [Résolu] Utiliser la configuration de phpstan dans avec grumphp

#3 05-05-2020 15:39:13

Re : [Résolu] Utiliser la configuration de phpstan dans avec grumphp

#4 05-05-2020 15:45:49

Re : [Résolu] Utiliser la configuration de phpstan dans avec grumphp

#5 05-05-2020 15:48:05

Re : [Résolu] Utiliser la configuration de phpstan dans avec grumphp

#6 05-05-2020 18:49:32

Re : [Résolu] Utiliser la configuration de phpstan dans avec grumphp

#7 05-05-2020 18:57:03

Re : [Résolu] Utiliser la configuration de phpstan dans avec grumphp

Pied de page des forums