marshmallow-code/marshmallow

Type hinting for `load` is too strict

Open

#1.457 aberto em 12 de nov. de 2019

Ver no GitHub
 (12 comments) (0 reactions) (0 assignees)Python (640 forks)batch import
help wantedtyping

Métricas do repositório

Stars
 (6.787 stars)
Métricas de merge de PR
 (Mesclagem média 4h 13m) (6 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
O problema é sobre a dica de tipo para o argumento `data` do método `load` no marshmallow. A dica atual usa `typing.Mapping`, o que está incorreto quando `many=True`, pois `data` é uma lista. Examine a definição do método `load` no código fonte (provavelmente em `marshmallow/schema.py`) e considere usar `typing.Union[Mapping, list]` ou assinaturas sobrecarregadas. A discussão nos comentários pode fornecer mais contexto sobre a abordagem preferida.
Pilha de tecnologia
python
Domain
backendapi
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
understanding of Python type hints
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70