vimeo/psalm

Notice in php 8.1 autoloading causes psalm to fatal error

Aperta

#7691 aperta il 18 feb 2022

Vedi su GitHubGuida contributor
 (11 commenti) (2 reazioni) (0 assegnatari)PHP (668 fork)batch import
Help wantedenhancement

Metriche repository

Star
 (5369 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga il gestore di errori in src/Psalm/Internal/ErrorHandler.php:66 per capire come gestisce attualmente gli avvisi. Determina se si possa aggiungere una semplice soppressione degli avvisi di deprecazione o un attributo di compatibilità. Esamina i commenti collegati per potenziali soluzioni.
Tech stack
php
Dominio
developer experiencetooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitPHPComposer
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70