marshmallow-code/marshmallow

Type hinting for `load` is too strict

Open

#1.457 geöffnet am 12. Nov. 2019

Auf GitHub ansehen
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (640 Forks)batch import
help wantedtyping

Repository-Metriken

Stars
 (6.787 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 4h 13m) (6 gemergte PRs in 30 T)

Beschreibung

The type hinting for load's data argument is set to typing.Mapping which makes sense for the many=False case, but for many=True case the type of data is list, which isn't a Mapping type.

Contributor Guide

Research-Richtung
Das Problem betrifft den Typ Hinweis für das `data` Argument der `load` Methode in marshmallow. Der aktuelle Hinweis verwendet `typing.Mapping`, was falsch ist, wenn `many=True`, da `data` dann eine Liste ist. Schauen Sie sich die Definition der `load` Methode im Quellcode (wahrscheinlich in `marshmallow/schema.py`) an und erwägen Sie die Verwendung von `typing.Union[Mapping, list]` oder überladenen Signaturen. Die Diskussion in den Kommentaren könnte weiteren Kontext zum bevorzugten Ansatz liefern.
Tech Stack
python
Domain
backendapi
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.
Unter 1 Stunde
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
understanding of Python type hints
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70