marshmallow-code/marshmallow

Type hinting for `load` is too strict

Open

#1457 aperta il 12 nov 2019

Vedi su GitHub
 (12 commenti) (0 reazioni) (0 assegnatari)Python (640 fork)batch import
help wantedtyping

Metriche repository

Star
 (6787 star)
Metriche merge PR
 (Merge medio 4h 13m) (6 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema riguarda il suggerimento di tipo per l'argomento `data` del metodo `load` in marshmallow. L'attuale suggerimento utilizza `typing.Mapping`, che è errato quando `many=True` poiché `data` è una lista. Esamina la definizione del metodo `load` nel codice sorgente (probabilmente in `marshmallow/schema.py`) e considera l'uso di `typing.Union[Mapping, list]` o di firme sovraccaricate. La discussione nei commenti potrebbe fornire ulteriore contesto sull'approccio preferito.
Tech stack
python
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
understanding of Python type hints
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70