mapbox/mapbox-gl-redux

Remove special case code

Offen

#9 geöffnet am 18.05.2018

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (3 Forks)github user discovery
good first issue

Repository-Metriken

Stars
 (37 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

There's functionality that made sense in Mapbox Studio but less so here as a broad utility:

https://github.com/mapbox/mapbox-gl-redux/blob/435c286e6397ff0febb59a94b4558ffbc22a3ad1/src/index.js#L38-L47

If there isn't a good case for keeping it around we should remove it.

Contributor Guide

Research-Richtung
Überprüfe die Zeilen 38-47 in src/index.js und stelle fest, ob die Sonderfall Funktionalität noch erforderlich ist; falls nicht, entferne den Code und stelle sicher, dass die Tests bestehen.
Tech Stack
javascript
Domain
frontend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90