-
Notifications
You must be signed in to change notification settings - Fork 415
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
feat(azure_functions): add azure functions integration #11474
base: main
Are you sure you want to change the base?
Conversation
|
7c49099
to
b9e3a70
Compare
try: | ||
client.get_ignored("/shutdown") | ||
except Exception: | ||
pass |
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.
🔴 Code Quality Violation
silent exception (...read more)
Using the pass
statement in an exception block ignores the exception. Exceptions should never be ignored. Instead, the user must add code to notify an exception occurred and attempt to handle it or recover from it.
The exception to this rule is the use of StopIteration
or StopAsyncIteration
when implementing a custom iterator (as those errors are used to acknowledge the end of a successful iteration).
b9e3a70
to
10a50a1
Compare
BenchmarksBenchmark execution time: 2024-11-27 16:23:15 Comparing candidate commit ac03dfd in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 387 metrics, 2 unstable metrics. scenario:iast_aspects-ospathsplitext_aspect
|
1fc8c18
to
12969c6
Compare
12969c6
to
fb17586
Compare
fb17586
to
cdecdaa
Compare
ca0a772
to
547dc32
Compare
This PR adds an integration for tracing the azure-functions package.
Additional Notes:
3.7
to3.11
(no3.12
support at the moment)linux/amd64
architecturemariadb
install was broken in the testrunner image so the install commands were updated in cc5b4a7Checklist
Reviewer Checklist