corejavascript/typeahead.js

Deduplication of common/utils

Open

#165 geöffnet am 10. Okt. 2017

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (232 Forks)batch import
enhancementhelp wanted

Repository-Metriken

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

Beschreibung

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?

Contributor Guide

Research-Richtung
Das Issue schlägt vor, 'common/utils' zu deduplizieren, das derzeit sowohl im Bloodhound als auch im Typeahead Modul kopiert ist. Untersuche das Verzeichnis 'common/utils' im Repository, um die gemeinsamen Hilfsfunktionen zu verstehen. Bewerte dann die beiden vorgeschlagenen Lösungen: Erstellung eines dritten gemeinsamen Moduls oder Aufteilung der Hilfsfunktionen in separate Klassen. Sieh dir die vorhandenen Moduldefinitionen in der Codebasis an, um zu verstehen, wie Abhängigkeiten verwaltet werden.
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.
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
JavaScriptmodule systems
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70