search

OPUS4 / opus4-common

Core package for OPUS 4.
Other
0 stars 5 forks source link

Namespace von "Opus\" auf "Opus\Common" umstellen #51

Closed j3nsch closed 2 years ago

j3nsch commented 2 years ago

Das Namespace für opus4-common soll auf "Opus\Common" umgestellt werden, damit jedes OPUS 4 Package, seinen eigenen Namespace hat. Momentan besteht die Möglichkeit, dass sich Namespaces in Common mit anderen Packages überlappen.

j3nsch commented 2 years ago

Für diese Umstellung müssen die Use-Statements in allen Package nachgezogen werden. Das ist also eine größere Aufgabe. Ich denke aber die klare Trennung der Namespaces der einzelnen Packages ist wichtig. Je früher wir diese Änderung vornehmen, desto weniger Arbeit. Die Zahl der Klassen in Common wird weiter wachsen, selbst schon für OPUS 4.7.2.

j3nsch commented 2 years ago

Für diese Umstellung sollen die Abhängigkeiten in den anderen Pakete auf den temporären Entwicklungsbranch von Common und anderen Packages gesetzt werden, so dass wir alle Pakete erfolgreich testen können, bevor wir die PRs übernehmen. Die Branches sollen commonNamespace heißen, in allen Repos. Alle Commits sollen mit OPUS4/opus4-common#51 (OPUS4/opus4-common#51) verknüpft werden.

j3nsch commented 2 years ago

Der Namespace von opus4-common ist umgestellt (auf dem Branch commonNamespace). Als nächstes mache ich die Umstellung in opus4-search, weil ich dort weitere Sachen ausprobieren will.

Für die Umstellung in Application sollte dort ein Issue angelegt werden, mit einer kurzen Beschreibung und einem Verweis auf OPUS4/opus4-common#51. In den anderen Repos kann das Issue hier verwendet werden. Application ist aber unser zentrales Repository für die Kommunikation mit der OPUS 4 Community. Die Issues dort sind sichtbarer und ich würde gerne die "großen" Änderungen dort in den Milestones sichtbar haben. Ich gehe davon aus, dass in Application einige hundert Dateien angepasst werden müssen.