marshmallow-code/marshmallow

Type hinting for `load` is too strict

Ouverte

#1 457 ouverte le 12 nov. 2019

Voir sur GitHubGuide contributeur
 (12 commentaires) (0 réaction) (0 personne assignée)Python (640 forks)batch import
help wantedtyping

Métriques du dépôt

Stars
 (6 787 étoiles)
Métriques de merge PR
 (Merge moyen 10h 55m) (7 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez l'indice de type pour le paramètre `data` dans la méthode `load` et déterminez comment le typer correctement pour accepter à la fois `Mapping` (pour les objets uniques) et `list` (pour les listes d'objets lorsque `many=True`). Envisagez d'utiliser `typing.Union` ou des surcharges.
Stack technique
python
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Pythontype hints
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65