-
Notifications
You must be signed in to change notification settings - Fork 613
feat(integrations): Add integration for aiomysql
#4703
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
base: master
Are you sure you want to change the base?
Changes from all commits
878dc5b
29b4e85
28e79b6
950032f
421abfa
ab35ef8
c09eb44
b3b04a1
d9d7282
faf1a87
89b8669
ad11c32
e25d3ad
5f26753
a7319c9
5d39b16
facbe7b
3d85885
6e05b78
308571f
a8e79d5
93a0de6
81825b6
a0143fd
53a276c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -66,6 +66,7 @@ def iter_default_integrations( | |
|
|
||
| _AUTO_ENABLING_INTEGRATIONS = [ | ||
| "sentry_sdk.integrations.aiohttp.AioHttpIntegration", | ||
| "sentry_sdk.integrations.aiomysql.AioMySQLIntegration", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We would wait until the integration has been tested over a few versions before adding the integration to be auto-enabled. Could you please remove the integration from auto-enabling list? 🙏 |
||
| "sentry_sdk.integrations.anthropic.AnthropicIntegration", | ||
| "sentry_sdk.integrations.ariadne.AriadneIntegration", | ||
| "sentry_sdk.integrations.arq.ArqIntegration", | ||
|
|
@@ -117,6 +118,7 @@ def iter_default_integrations( | |
|
|
||
| _MIN_VERSIONS = { | ||
| "aiohttp": (3, 4), | ||
| "aiomysql": (0, 1, 1), | ||
| "anthropic": (0, 16), | ||
| "ariadne": (0, 20), | ||
| "arq": (0, 23), | ||
|
|
||
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.
Please avoid changes in unrelated files 🙏