marshmallow-code/marshmallow

Type hinting for `load` is too strict

Aberta

#1.457 aberto em 12 de nov. de 2019

Ver no GitHubGuia do colaborador
 (12 comentários) (0 reação) (0 responsável)Python (640 forks)batch import
help wantedtyping

Métricas do repositório

Stars
 (6.787 estrelas)
Métricas de merge de PR
 (Mesclagem média 10h 55m) (7 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
Examine a dica de tipo para o parâmetro `data` no método `load` e determine como tipá lo corretamente para aceitar tanto `Mapping` (para objetos únicos) quanto `list` (para listas de objetos quando `many=True`). Considere usar `typing.Union` ou sobrecargas.
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.
1-3 horas
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.
Principalmente claro
Prerequisites
Pythontype hints
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65