Skip to content

refactor(translator): move into src layout and use uv properly#213

Merged
crankynetman merged 10 commits intomainfrom
topic/chriscummings/uv-workspaces
Feb 27, 2026
Merged

refactor(translator): move into src layout and use uv properly#213
crankynetman merged 10 commits intomainfrom
topic/chriscummings/uv-workspaces

Conversation

@crankynetman
Copy link
Collaborator

@crankynetman crankynetman commented Feb 25, 2026

Now translator is actually a proper python package. We could publish it in pypy if we were so inclined, but for now, at least it's using uv workspaces and actually is setup to not be so intermingled with scram-django. I also updated scheduler to now use the root lockfile so that we can use one lockfile to rule them all. We also have a pre-commit step to check for an out of date lockfile to make sure we don't change pyproject but forget to update the lock. Having the lock file committed allows us to have our builds actually be reproducible, which is nice.

Also updates django-simple-history as pkg_resources seems to have been broken and 3.4+ removes that dependency (and is still compatible with django 4.2)

@github-actions
Copy link

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 127270c

@github-actions
Copy link

github-actions bot commented Feb 27, 2026

Django Pytest/Behave Coverage

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

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against 127270c

Copy link
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.

looks great to me! consistency and getting away from the wild wild west is good.

@crankynetman crankynetman merged commit 5335350 into main Feb 27, 2026
18 of 20 checks passed
@crankynetman crankynetman deleted the topic/chriscummings/uv-workspaces branch February 27, 2026 19:25
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