Repository navigation
Include optional oxpull task coordination in django-ox - #137
Merged
Merged
Conversation
Include the optional `oxpull` app in the django-ox wheel and sdist under BSD-3-Clause. All features are free and open source. Provide unique tasks, batches, shared admission rate limits, and task workflows. The workflows API remains provisional. Require Django 6.0+ only when the app is enabled. Projects that do not enable it do not load it. Keep django-ox support for Django 5.2 through the django-tasks backport. Preserve settings, import paths, task data, the app label, table names, the five `oxpull` migrations, and persisted task paths. Align `oxpull.__version__` with `django_ox.__version__`. Earlier separate installs must uninstall the earlier `oxpull` distribution first, then upgrade django-ox, migrate, and restart. Upgrade guide: https://oxpull.com/django-ox/migrating/#earlier-oxpull-installs Add sharded app tests on Django 6.0+ with SQLite, PostgreSQL (plain and pooled), and MySQL. Run long-running hazard tests for relevant changed paths, and in full on release-candidate tags, nightly, and manual dispatch. Add opt-in installed-wheel upgrade tests and a single aggregate result check. Keep running the existing django-ox suite on Django 5.2 to 6.1. Add and extend tests for the Django 5.2 refusal of `oxpull`, CI path selection, release checks, log event documentation, and docs index page URLs. Add type-check settings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the optional
oxpullapp to django-ox's wheel and sdist under BSD-3-Clause. Previously distributed separately, all its features are now free and open source:See the new Task coordination docs in
docs/features/in this PR.To enable it on Django 6.0+, keep
django_oxinINSTALLED_APPS, addoxpull, set the task backend tooxpull.backend.OxpullBackend, and runmigrate. Projects that do not enable the app do not load it. django-ox still supports Django 5.2 through the django-tasks backport.Settings, import paths, task data, app label, table names, the five
oxpullmigrations, and persisted task paths stay unchanged. Earlier separate installs must uninstall the earlieroxpulldistribution first, then upgrade django-ox, migrate, and restart. Follow the upgrade guide in the "Earlieroxpullinstalls" section ofdocs/migrating.mdin this PR.Reviewers can check:
Django 5.2 to 6.1, with added and extended tests for the Django 5.2
refusal of
oxpull, CI path selection, release checks, log eventdocumentation, and docs index page URLs, plus type-check settings.