corejavascript/typeahead.js

Deduplication of common/utils

Open

#165 aperta il 10 ott 2017

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (232 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (956 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Up until version 0.10.x, common/utils was shared by Bloodhound & Typeahead.

From version 0.11.x onwards, Bloodhound & Typeahead are defined as UMD modules, and each module has its own copy of common/utils. This results in unnecessary redundancy / bloat.

There's several solutions for this. Personally, I lean towards one of these options :

  • Put common/utils into a third module that is used as a dependency by both Bloodhound & Typeahead
  • Replace common/utils by two distinct util classes that are no longer shared between Bloodhound & Typeahead

What do you guys think about this?

Guida contributor

Direzione di ricerca
Il problema propone di deduplicare 'common/utils', attualmente copiato sia nei moduli Bloodhound che Typeahead. Indaga la directory 'common/utils' nel repository per comprendere le utilità condivise. Quindi valuta le due soluzioni proposte: creare un terzo modulo condiviso o suddividere le utilità in classi distinte. Esamina le definizioni dei moduli esistenti nella codebase per vedere come vengono gestite le dipendenze.
Tech stack
javascript
Dominio
frontend
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
JavaScriptmodule systems
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70