-
-
Notifications
You must be signed in to change notification settings - Fork 940
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create Django Transport #2228
base: main
Are you sure you want to change the base?
Create Django Transport #2228
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2228 +/- ##
==========================================
+ Coverage 81.50% 81.63% +0.12%
==========================================
Files 77 82 +5
Lines 9524 9642 +118
Branches 1152 1158 +6
==========================================
+ Hits 7763 7871 +108
- Misses 1569 1579 +10
Partials 192 192 ☔ View full report in Codecov by Sentry. |
357ecd5
to
3fd61a0
Compare
2b6ac41
to
4df3ff8
Compare
136aeb1
to
cec7b95
Compare
35f0c28
to
c822688
Compare
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I accept the premise of this PR. but I want to take some time to properly review it
Add Django ORM transport. This coincides with celery/celery#9500
Tested with
sqlite
andpostgres
. Also,flower
is working reasonably well.This effort is inspired by Oban for Elixir and River for Go.