mapbox/mapbox-gl-redux

Remove special case code

Aperta

#9 aperta il 18 mag 2018

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (3 fork)github user discovery
good first issue

Metriche repository

Star
 (37 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina le righe 38-47 in src/index.js e verifica se la funzionalità del caso speciale è ancora necessaria; in caso contrario, rimuovi il codice e assicurati che i test passino.
Tech stack
javascript
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90