Skip to content

[DX] Added django-debug-toolbar as a dev dependency#956

Merged
eepMoody merged 2 commits into
open5e:stagingfrom
calumbell:881/django-debug
Jun 13, 2026
Merged

[DX] Added django-debug-toolbar as a dev dependency#956
eepMoody merged 2 commits into
open5e:stagingfrom
calumbell:881/django-debug

Conversation

@calumbell

@calumbell calumbell commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds the Django Debug Toolbar to our list of development dependencies. This package has been extremely helpful in identifying N+1 errors on the API and is well worth having installed as default.

How was this tested

  • (Local) Changing OPEN5E_DEBUG environment variable to False to confirm that the toolbar didn't run outside of debug mode.
  • (Local) Run pytest to make sure the toolbar didn't interfere with it.

Pending to see whether the the toolbar is able to get out of the way correctly during CI tests. Standby.

@calumbell calumbell closed this Jun 9, 2026
@calumbell calumbell reopened this Jun 9, 2026
@calumbell

Copy link
Copy Markdown
Contributor Author

Alright, looks like CI is failing. After a brief investigation the problems appears to be on the Docker image build step:

#10 ERROR: process "/bin/sh -c uv sync --frozen --no-dev --no-install-project" did not complete successfully: exit code: 2
------
 > [builder 4/7] RUN uv sync --frozen --no-dev --no-install-project:
0.184 error: Required uv version `>=0.10` does not match the running version `0.9.30`
------

Looks like some kind of uv version mismatch. Will investigate further later this week.

@eepMoody eepMoody merged commit d4276c5 into open5e:staging Jun 13, 2026
1 check passed
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