-
Notifications
You must be signed in to change notification settings - Fork 278
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
manual fixes #2455
manual fixes #2455
Conversation
5ec9dce
to
47c85fe
Compare
Codecov Report
@@ Coverage Diff @@
## main #2455 +/- ##
==========================================
- Coverage 62.10% 62.08% -0.02%
==========================================
Files 121 121
Lines 22833 22830 -3
Branches 11212 11212
==========================================
- Hits 14181 14175 -6
- Misses 6445 6448 +3
Partials 2207 2207
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
clang-tidy suggests this but fails to apply it. Signed-off-by: Rosen Penev <rosenp@gmail.com>
Signed-off-by: Rosen Penev <rosenp@gmail.com>
Fallout from recently commited external inireader. |
Signed-off-by: Rosen Penev <rosenp@gmail.com>
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.
I am not sure if it is related to the changes you did, but the fuzzer job failed. Can you actually retrigger those CI jobs? Otherwise I can do it for you.
Signed-off-by: Rosen Penev <rosenp@gmail.com>
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, Thanks!
No description provided.