vimeo/psalm

Notice in php 8.1 autoloading causes psalm to fatal error

Ouverte

#7 691 ouverte le 18 févr. 2022

Voir sur GitHubGuide contributeur
 (11 commentaires) (2 réactions) (0 personne assignée)PHP (668 forks)batch import
Help wantedenhancement

Métriques du dépôt

Stars
 (5 369 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

If any autoloaded files (in libraries) fire the notice:

Return type of Class:method() should either be compatible with Class::method: OtherClass, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in FILE:LINE in /app/vendor/vimeo/psalm/src/Psalm/Internal/ErrorHandler.php:66

then psalm fails to run any analysis at all. Obviously, libraries should update their code, but psalm should still be usable.

Psalm is installed via composer at version 4.20.0

Guide contributeur

Direction de recherche
Étudiez le gestionnaire d'erreurs dans src/Psalm/Internal/ErrorHandler.php:66 pour comprendre comment il gère actuellement les notices. Déterminez si une simple suppression des notices d'obsolescence ou un attribut de compatibilité pourrait être ajouté. Examinez les commentaires liés pour trouver des solutions potentielles.
Stack technique
php
Domaine
developer experiencetooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPHPComposer
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70