dvdciri/daggraph

Add constructor injected dependencies to the graph

Offen

#53 geöffnet am 19.08.2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (59 Forks)batch import
featurehelp wanted

Repository-Metriken

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

Beschreibung

Daggraph need to be able to recognise constructor injected dependencies and add them to the graph.

There are some open questions here:

  • Where should we put the class which is constructor injecting some dependencies?
  • Where should we put the injected dependencies?

This is the regex for koltin: /(?:class)\s(\w*).*?(?:@Inject).*?(?:constructor)\((.*?)\)\s?\{/gs This is the regex for java: /(?:@Inject).*?\s(\w*?)\((.*?)\)\s?{/gs

Contributor Guide

Research-Richtung
Untersuche das bestehende Graphenmodell und die Parsing Logik in Daggraph. Verstehe, wie derzeit Klassen und Abhängigkeiten extrahiert werden. Erweitere dann die Regex Muster, um die Konstruktorinjektion sowohl in Kotlin als auch in Java zu erfassen, wie angegeben. Entscheide über die Platzierung im Graphen (z. B. Knoten für die injizierte Klasse, Kanten für injizierte Abhängigkeiten). Sieh dir ähnliche vorhandene Funktionen als Anleitung an.
Tech Stack
javascriptkotlinjava
Domain
tooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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
Dependency injectionRegexDaggraph codebase
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65