facundoolano/google-play-scraper

Don't mutate options objects

Aberta

#238 aberto em 2 de jun. de 2018

Ver no GitHub
 (1 comentário) (0 reação) (0 responsável)JavaScript (723 forks)github user discovery
buggood first issue

Métricas do repositório

Stars
 (2.948 estrelas)
Métricas de merge de PR
 (Mesclagem média 103d 2h) (4 fundiu PRs em 30d)

Description

Most of the methods take an options object and then mutate by setting default values, like here. This is not a good practice, since the methods don't own those objects and are introducing unexpected side effects. Use Object.assign instead to make copies of the options.

Guia do colaborador

Direção de pesquisa
Identifique todos os métodos no código que mutam o objeto de parâmetros options. Para cada um, substitua a mutação por uma cópia usando Object.assign ou sintaxe de espalhamento para evitar efeitos colaterais.
Pilha de tecnologia
javascriptnodejs
Domain
backendapi
Tipo Issue
Refatoração
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.
Claro
Prerequisites
JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80