-
Notifications
You must be signed in to change notification settings - Fork 32
Open
Description
I recently used the django-ida cookiecutter. Here are various problems I encountered that we should fix, some of which are in the django-ida cookiecutter and some of which are in the higher level python-template:
- There are two versions of
pull_translations, one old-style and a newer one using Atlas. See theOPENEDX_ATLAS_PULLin the Makefile. Atlas is no longer experimental, and now should be default, and the oldpull_translationsshould be removed. - Remove devstack references and provisioning script
- Remove Dockerfile and references (including docker-publish workflow)
- Wrong Python version in tox reference -- should match Python version in tox.ini, and installed in workflow
- Remove dependency on edx-django-release-util
- Remove
ALLOWED_HOSTS = ['*']or add comment explaining why it is necessary. (This is a security issue.) - Remove requirements.txt
- Remove PyPI publish workflow from django-ida template -- we don't publish IDAs to PyPI
- Add
make compile-requirements(see edx-platform) - Switch from
PIP_COMPILE_OPTStoCOMPILE_OPTS-- pip pays attention to allPIP_-prefixed env vars, and we should stay out of that namespace. - Allow chaining
COMPILE_OPTS(accept extensions to it from command line) - Get rid of
clean_tox(tox is supposed to clean itself) - Healthcheck docstring needs trailing slash on the sample call to avoid redirects
- Healthcheck docstring shouldn't use edx.org example domain
- Update default manage.py from Django's own template (I think that's https://github.com/django/django/blob/4.2.20/django/conf/project_template/manage.py-tpl) -- we've got an old one. Look into other things that should be updated around this as well.
- Override CHANGELOG.rst to not have version strings in headings
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels