-
Notifications
You must be signed in to change notification settings - Fork 58
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
libsexpp updated to 0.8.8 #2242
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2242 +/- ##
==========================================
- Coverage 83.58% 83.58% -0.01%
==========================================
Files 107 107
Lines 23174 23167 -7
==========================================
- Hits 19371 19364 -7
Misses 3803 3803 ☔ View full report in Codecov by Sentry. |
854201a
to
d643129
Compare
d643129
to
0d4c203
Compare
ae69bf5
to
8fe8b89
Compare
cb0b04f
to
c02e22c
Compare
lgtm. There is still https://github.com/rnpgp/rnp/blame/main/src/lib/types.h#L75 I guess one needs to include the |
133ebf8
to
817caea
Compare
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.
@maxirmx thanks, please merge when ready.
@maxirmx Yeah, like a bunch of other PRs :) Waiting for the second approval cc @ronaldtse |
@maxirmx Now PR with CI fixes is merged, could you please rebase this? Or I can do that as well. |
817caea
to
e0790a2
Compare
Closes #2241