-
-
Notifications
You must be signed in to change notification settings - Fork 944
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.54% 81.67% +0.12%
==========================================
Files 77 82 +5
Lines 9517 9635 +118
Branches 1154 1160 +6
==========================================
+ Hits 7761 7869 +108
- Misses 1564 1574 +10
Partials 192 192 ☔ View full report in Codecov by Sentry. |
357ecd5
to
3fd61a0
Compare
2b6ac41
to
4df3ff8
Compare
136aeb1
to
cec7b95
Compare
c822688
to
0986639
Compare
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
for more information, see https://pre-commit.ci
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.