-
Notifications
You must be signed in to change notification settings - Fork 593
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
Handle CESQL parsing panics gracefully #6140
Conversation
6d04940
to
2888e86
Compare
Codecov Report
@@ Coverage Diff @@
## main #6140 +/- ##
==========================================
- Coverage 82.09% 82.06% -0.03%
==========================================
Files 231 231
Lines 7765 7769 +4
==========================================
+ Hits 6375 6376 +1
- Misses 940 942 +2
- Partials 450 451 +1
Continue to review full report at Codecov.
|
Signed-off-by: Ahmed Abdalla <aabdelre@redhat.com>
2888e86
to
38b1eed
Compare
The following is the coverage report on the affected files.
|
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: devguyio, matzew The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Proposed Changes
Pre-review Checklist
Release Note