search

Romern / syncMyMoodle

Synchronization client for RWTH Moodle
GNU General Public License v3.0
72 stars 18 forks source link

Split __main__.py across different files #97

Open n0toose opened 2 years ago

n0toose commented 2 years ago

If everything is under one file, syncMyMoodle will keep getting progressively harder to maintain and modify.

A better approach would be keeping the core functionality of main() under __main__.py and putting things like class definitions to other files. The goal here would be making the code even more "OOP"-like than it already is and improve modularity. This should help with developing additional extensions in the long run.

septatrix commented 2 years ago

This is already partially done on the v2 branch (on which I haven't worked in ages). My eventual goal which is not yet implemented on v2 would be to provide a decorator which can register handers for each module and pass in the session

n0toose commented 2 years ago

Is it possible for us to eventually apply some of your work onto the main branch?