search

arbeitsgruppe-digitale-altnordistik / Sammlung-Toole

A new look on Handrit.is data
https://arbeitsgruppe-digitale-altnordistik.github.io/Sammlung-Toole/
MIT License
0 stars 0 forks source link

overhaul repo structure #100

Closed BalduinLandolt closed 2 years ago

BalduinLandolt commented 2 years ago

@kraus-s I knew I had given this some thought, but couldn't find my notes in any of the issues... turns out it was in a WhatsApp conversation. xD
I just copy it here for the record (and so I don't have to search for it again), but we'll have to see how we want the structure - and how streamlit wants the structure for a multi-page setup...

[wollen wir,] sobald dein PR gemerged ist, aber bevor du einen nächsten startest, die Repo-Struktur etwas umbauen? Mir schwebt von der makrostruktur ungefär sowas vor: https://realpython.com/python-application-layouts/#application-with-internal-packages also einfach, dass wir allen code in einem unterordner haben (kann meinetwegen auch src/ heissen). Ausserdem sollte interface.py im gui/ ordner sein; und ein grossteil von dem, was in unseren utils ist, ist eigentlich nicht utility, sondern sollte in einen lib ordner, weil es einfach unsere eigene library ist. (files verschieben gibt immer so furchtbar viele unnötige konflikte, wenn PRs offen sind... darum wäre es super, wenn ich dir das für zwischen den PRs delegieren könnte.) Wenn das gemacht ist, kann ich mich mal ans schreiben der unit test machen - das wird dann auch vieles erleichtern.

BalduinLandolt commented 2 years ago

Done, thanks @kraus-s !