DIRACGrid / diracx

The neXt DIRAC generation
GNU General Public License v3.0
8 stars 19 forks source link

Re-organize the code #268

Open chaen opened 2 months ago

chaen commented 2 months ago

The __init__.py contains too much code, but we agree we want to avoid the extreme of one class per file (from JobDB import JobDB style). The current consensus would be like that:

fstagni commented 2 weeks ago

Right now, I see that https://github.com/DIRACGrid/diracx/blob/main/diracx-routers/src/diracx/routers/job_manager/__init__.py contains the code of what in DIRAC was JobManager and JobMonitor handlers. The only other WMS related router created is https://github.com/DIRACGrid/diracx/blob/main/diracx-routers/src/diracx/routers/job_manager/sandboxes.py.

Do I understand correctly that you would prefer to re-organize in a way similar to what is done for auth ? (https://github.com/DIRACGrid/diracx/blob/main/diracx-routers/src/diracx/routers/auth/__init__.py)