-
Notifications
You must be signed in to change notification settings - Fork 395
DI: notify about failed probe condition evaluation, with rate limiting #4979
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?
Conversation
Typing analysisIgnored filesThere are 539 ignored files in the Steepfile out of 842. Ignored files
Note: Ignored files are excluded from the next sections.
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage 🔗 Commit SHA: 4c56456 | Docs | Was this helpful? Give us feedback! |
What does this PR do?
Adds notifications about failed condition evaluation for line and method probes. The notifications are subject to a fixed rate limit of one per second.
Motivation:
Feature parity with other languages
Change log entry
None (expression language is not yet released to customers)
Additional Notes:
How to test the change?
Tests are added