marshmallow-code/marshmallow

Type hinting for `load` is too strict

Offen

#1.457 geöffnet am 12.11.2019

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

Repository-Metriken

Stars
 (6.787 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 10h 55m) (7 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
Untersuche den Typ Hinweis für den Parameter `data` in der `load` Methode und bestimme, wie man ihn korrekt typisiert, um sowohl `Mapping` (für einzelne Objekte) als auch `list` (für Listen von Objekten bei `many=True`) zu akzeptieren. Erwäge die Verwendung von `typing.Union` oder Überladungen.
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.
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.
Meist klar
Voraussetzungen
Pythontype hints
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65