Skip to content

refactor(django): Use UV Workspaces and src-layout for scram django app#217

Open
crankynetman wants to merge 17 commits intomainfrom
topic/chriscummings/uv-workspaces-django
Open

refactor(django): Use UV Workspaces and src-layout for scram django app#217
crankynetman wants to merge 17 commits intomainfrom
topic/chriscummings/uv-workspaces-django

Conversation

@crankynetman
Copy link
Copy Markdown
Collaborator

Good gravy this looks worse than it is. The big changes are setup.cfg being split into the parts needed in various pyproject files. That and since I touched everything, ruff wants to reformat it all since the line limit of 119 was apparently being ignored before.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 26, 2026

Pytest Celery Scheduler Coverage

File Coverage
All files 100%

Minimum allowed coverage is 80%

Generated by 🐒 cobertura-action against 4f5f254

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 27, 2026

Django Pytest/Behave Coverage

File Coverage
All files 78%
src/config/consumers.py 59%
src/config/urls.py 69%
src/config/settings/base.py 69%
src/config/settings/local.py 72%
src/scram/route_manager/admin.py 71%
src/scram/route_manager/models.py 79%
src/scram/route_manager/views.py 88%
src/scram/route_manager/api/serializers.py 92%
src/scram/route_manager/api/views.py 63%
src/scram/shared/shared_code.py 56%
src/scram/templates/403.html 93%
src/scram/templates/404.html 93%

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against 4f5f254

Comment thread django/src/config/settings/base.py
Copy link
Copy Markdown
Collaborator

@samoehlert samoehlert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a honker of a PR, but this is good work breaking down the project into more distinct parts. Big win.

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