vimeo/psalm

Improve support for is_file and family

Open

#7.770 geöffnet am 10. März 2022

Auf GitHub ansehen
 (13 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)PHP (668 Forks)batch import
Help wantedeasy problemsenhancementgood first issueinternal stubs/callmap

Repository-Metriken

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

Beschreibung

Hi,

The functions that expect a valid path (file_exists, is_readable, ...) always return false when an empty string is provided ( https://3v4l.org/TcecN ). So when those functions return true we can say that the argument is a non-empty string.

If my assumption is right it could help with situations like this: https://psalm.dev/r/26f209fe21

Contributor Guide

Research-Richtung
Untersuche die interne Handhabung von Dateisystemfunktionen wie file exists und is readable in Psalm. Das Problem deutet darauf hin, dass, wenn diese true zurückgeben, das Argument ein nicht leerer String ist. Überprüfe den Playground Link (https://psalm.dev/r/26f209fe21) für ein konkretes Beispiel. Mögliche Änderungen könnten die Anpassung des Rückgabetyps oder das Hinzufügen einer Typverengungsregel in den internen Funktionsdefinitionen von Psalm umfassen.
Tech Stack
php
Domain
developer experience
Issue Type
Funktion
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.
60