Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore/build refactor: Add django-debug-toolbar and separate debug settings #908

Merged
merged 8 commits into from
Sep 23, 2024

Conversation

dchiller
Copy link
Collaborator

@dchiller dchiller commented Sep 20, 2024

This PR completes a variety of chores and refactorings related to debugging. Once merged, the Cantus Ultimus django application will be able to make use of VSCode's debugging system and the django-debug-toolbar utility.

This PR:

  • updates submodule references for the production-mei-files submodule (and test mei files from that submodule)
  • updates packaged for debugging (debugpy - used by the VS Code built-in python debugger) and type checking for the lxml package
  • separates django settings for production and debug to allow for simpler use of separate settings for both + testing

configure urls.py and settings.py to allow the toolbar to show
when DEBUG = True
- Separates out development/debugging django settings into a separate
debug_settings.py file
- Adds ability to set a non-default (non cantusdata.settings) django
settings file via environment variables
- Creates a compose-dev.yaml compose configuration file for development
- Removes unnecessary port mappings from the production docker compose
configuration
- Updates the README to account for these changes
app/public/cantusdata/debug_settings.py Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
dchiller and others added 2 commits September 23, 2024 12:17
Co-authored-by: Lucas <71031342+lucasmarchd01@users.noreply.github.com>
@dchiller dchiller merged commit b4798e3 into DDMAL:main Sep 23, 2024
2 checks passed
@dchiller dchiller deleted the debug-settings-config branch September 23, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants