rdmorganiser / rdmo

A tool to support the planning, implementation, and organization of research data management.
https://rdmorganiser.github.io
Apache License 2.0
105 stars 49 forks source link

Add rdmo-admin script #1185

Closed jochenklar closed 6 days ago

jochenklar commented 2 weeks ago

This PR adds a rdmo-admin script, which works like the django-admin script and offers a way to use management scripts outside an rdmo-app (e.g. in the rdmo repo).

(My) use cases are:

rdmo-admin npm run install
rdmo-admin npm run watch
...

instead of

nvm use
npm run watch
...

or

rdmo-admin build

instead of

python -m build

which might not be to impressive, but has a more uniform interface and might be easier to document.

I think this could really be helpful if we extent it to help with the setup of RDMO, something like

pip install rdmo
rdmo-admin init

We could even think about replacing rdmo-app with a cookiecutter, but I am not sure about this, yet.

Another use case would be to simplify the makemessages step.

Any other ideas are welcome. Also, do we need this or is this additional complexity.

coveralls commented 2 weeks ago

Coverage Status

coverage: 90.409% (-0.07%) from 90.478% when pulling b46853f9817799df1d05525ab77be73f81b0e290 on rdmo_admin into 1eadee51947399601d85e476b261391826e0f95a on main.

MyPyDavid commented 2 weeks ago

rdmo-admin startapp ?? 😅

jochenklar commented 2 weeks ago

Maybe we should combine commands, build could run npm run build:prod and python -m build, while watch just runs npm run watch.

jochenklar commented 1 week ago

After some thoughts I am wondering if this only solves my problems and could be better solved with a local custom script or Makefile. We can discuss this in our next meeting.

MyPyDavid commented 1 week ago

I would keep it, think it's also useful for the local dev setup..

jochenklar commented 1 week ago

Rebased and ready (after tests are ok).