-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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] sentry: Fix DeprecationWarning for sentry_sdk.push_scope #3106
base: 16.0
Are you sure you want to change the base?
[FIX] sentry: Fix DeprecationWarning for sentry_sdk.push_scope #3106
Conversation
Hi @barsi, @versada, @naglis, @moylop260, @fernandahf, |
@hnavarro-kernet could you rebase? |
7f6c3d6
to
5fe496e
Compare
Should be rebased now @MiquelRForgeFlow |
5fe496e
to
0fe7d3a
Compare
Pre-commit successful, waiting for |
tests are red, please, could you check it? |
0fe7d3a
to
d239890
Compare
d239890
to
d39f13a
Compare
All 4 PRs depend on each other I guess:
So maybe merge them into one? Maybe backport from this one that should have fixed all the issues? If a maintainer with approval rights want to tell me what to do I'd be happy to, if not, this has been on prod since the PR was opened so I'm not going to fight the tests. |
See: https://docs.sentry.io/platforms/python/migration/1.x-to-2.x\#scope-pushing