facundoolano/google-play-scraper

Don't mutate options objects

Offen

#238 geöffnet am 02.06.2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (723 Forks)github user discovery
buggood first issue

Repository-Metriken

Stars
 (2.948 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 103T 2h) (4 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Identifiziere alle Methoden im Code, die das Optionsobjekt verändern. Ersetze für jede Methode die Mutation durch eine Kopie mit Object.assign oder Spread Syntax, um Seiteneffekte zu vermeiden.
Tech Stack
javascriptnodejs
Domain
backendapi
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80