facundoolano/google-play-scraper

Don't mutate options objects

Aperta

#238 aperta il 2 giu 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (723 fork)github user discovery
buggood first issue

Metriche repository

Star
 (2948 stelle)
Metriche merge PR
 (Merge medio 103g 2h) (4 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Identifica tutti i metodi nel codice che mutano l'oggetto options passato come parametro. Per ciascuno, sostituisci la mutazione con una copia usando Object.assign o lo spread operator per evitare effetti collaterali.
Tech stack
javascriptnodejs
Dominio
backendapi
Tipo issue
Refactoring
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.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80