marshmallow-code/marshmallow

Type hinting for `load` is too strict

Aperta

#1457 aperta il 12 nov 2019

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

Metriche repository

Star
 (6787 stelle)
Metriche merge PR
 (Merge medio 10h 55m) (7 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
Esamina l'hint di tipo per il parametro `data` nel metodo `load` e determina come tipizzarlo correttamente per accettare sia `Mapping` (per oggetti singoli) che `list` (per elenchi di oggetti quando `many=True`). Considera l'uso di `typing.Union` o overload.
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.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Pythontype hints
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65