Add rdmo-admin script #1185
Open
Add rdmo-admin script #1185
+146
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
|
Maybe we should combine commands, |
|
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. |
|
I would keep it, think it's also useful for the local dev setup.. |
jochenklar
force-pushed
the
rdmo_admin
branch
from
b46853f
to
cba45de
Compare
|
Rebased and ready (after tests are ok). |
MyPyDavid
reviewed
Nov 18, 2024
MyPyDavid
reviewed
Nov 18, 2024
| def clean_dir(self, path, quiet=False): | ||
| if path and os.path.exists(path): | ||
| shutil.rmtree(path) | ||
| if not quiet: |
There was a problem hiding this comment.
where does quiet come from? it's not in the command arguments
There was a problem hiding this comment.
To not "log" removal of the __pycache__ directories, see L37.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a
rdmo-adminscript, which works like thedjango-adminscript and offers a way to use management scripts outside anrdmo-app(e.g. in therdmorepo).(My) use cases are:
instead of
or
instead of
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
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
makemessagesstep.Any other ideas are welcome. Also, do we need this or is this additional complexity.