-
Notifications
You must be signed in to change notification settings - Fork 28
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
fix: Use new func name in sentry-sdk celery config #997
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. 📢 Thoughts on this report? Let us know! |
✅ All tests successful. No failed tests were found. 📣 Thoughts on this report? Let Codecov know! | Powered by Codecov |
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #997 +/- ##
==========================================
- Coverage 96.03% 96.03% -0.01%
==========================================
Files 828 828
Lines 19166 19156 -10
==========================================
- Hits 18407 18397 -10
Misses 759 759
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This PR includes changes to |
This PR requires some whack-a-mole with deps versions. We'll handle downgrading shared to be compatible with api instead of the other way around with this pr - codecov/shared#432 |
It appears we have custom configuration of sentry-sdk to address this issue with celery - getsentry/sentry-python#1416
When bumping to the latest version of sentry-sdk, we need to rename the function
_wrap_apply_async
-->_wrap_task_run