Update django-q with django-q2 #739
Draft
Conversation
This file contains 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
* refactor: show function name as it runs * feat: log func name when processed Co-authored-by: Noortheen Raja <[email protected]>
Migrate codecov to coveralls
* Fix readthedocs file * remove trigger from test workflow
* Update django docs links to 4.0 * Update Django Q to Django Q2 * Use different theme to make difference more clear between the original and this forked one * Added info notice about this fork * Some small layout/typo fixes
Co-authored-by: ethemguner
Co-authored-by: Marc Sabatier <[email protected]>
Co-authored-by: Marc Sabatier <[email protected]>
* Add the group column to OrmQ admin page * Add group field to tasks list and search_field * Add search by name to Schedule admin page * Limit succesful task history per schedule with SAVE_LIMIT_GROUP config * Improve ScheduleAdmin list view performance Co-authored-by: Marc Sabatier <[email protected]>
* feat: option to save tasks per group/func/name * test: update tests for save limit * fix: convert func when save-limit checking when passed as function it needs to be converted to work * Add warning if option is not valid Co-authored-by: Noortheen Raja <[email protected]>
Instead of checking if the django-q logger has any handlers directly, use the hasHandlers [1] method, which will check if any parent loggers have handlers as well. This fixes a bug where django-q configures it's own logging even though the root logger has a handler set 1. https://docs.python.org/3/library/logging.html#logging.Logger.hasHandlers Co-authored-by: Wesley Ellis <[email protected]>
Co-authored-by: Rajeev KL <[email protected]>
* fix django q2 install command and remove space * remove warnings for very old versions
having build-system config helps installing this without setup.py built
…ix (#18) task names in logs
* Replace black/isort with ruff * fix typo
* add example project * format * fix format
* Add index on task table * also add models file * add make commands and limit index * format
* Add twine check * add poetry to install * move up remove black/flake formatting * fix path * fix? * fix rst? * add rstcheck * test all files? * all * remove complaining image * only check readme * remove not needed dependencies
* fix prometheus integration * fix prometheus integration * Add also lowercase option As they seem to support that too: https://github.com/prometheus/client_python/blob/d7c9cd88c7f50097cd86869974301df7615bc9c0/prometheus_client/multiprocess.py#L166-L167 * format --------- Co-authored-by: Stan Triepels <[email protected]>
* Update monitor.py * format --------- Co-authored-by: Stan Triepels <[email protected]>
* Fix 'receive_message_wait_time_seconds' SQS broker management * Remove socket volume from localstack container
The file should not be installed directly in site-packages. Use the correct include syntax to include them into the source distribution package only.
- avoid extra queries when checking the count of success in db
* delete old code * remove imports
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Still a work in progress. What needs to be done:
django-q(instead ofdjango-q2) everywhere. PR: Docs updates django-q2/django-q2#163django-q2to push both packages to PyPI. (disable push when on the django-q repo)After merge: