vimeo/psalm

Bug: psalm.xml cannot be loaded if a non-existent directory is specified

Open

#7.111 geöffnet am 9. Dez. 2021

Auf GitHub ansehen
 (2 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)PHP (668 Forks)batch import
Help wantedenhancement

Repository-Metriken

Stars
 (5.369 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 3T 12h) (5 gemergte PRs in 30 T)

Beschreibung

I just had this strange error

Problem parsing /var/www/html/psalm.xml: Could not resolve config path to /var/www/html/

thrown from here.

The cause was this snippet in my psalm.xml:

        <RedundantConditionGivenDocblockType>
            <errorLevel type="suppress">
                <directory name="bar/"/>
            </errorLevel>
        </RedundantConditionGivenDocblockType>

I just had moved bar to foo/bar. So it could not resolve bar (realpath in Line 168 returned false). That was hard to find because the error was not very informative. Maybe we could change that so that I can see the real mistake a bit more clearly? 😅

Contributor Guide

Research-Richtung
Verbessere die Fehlermeldung in FileFilter.php, wenn realpath false zurückgibt, sodass der nicht vorhandene Verzeichnisname angezeigt wird.
Tech Stack
php
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PHP
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70