-
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
Refactor signature subpackets for better code and update FFI. #2281
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2281 +/- ##
==========================================
+ Coverage 84.27% 84.65% +0.38%
==========================================
Files 114 116 +2
Lines 23324 23422 +98
==========================================
+ Hits 19656 19829 +173
+ Misses 3668 3593 -75 ☔ View full report in Codecov by Sentry. |
613fd18
to
5ddd706
Compare
903fddf
to
0df336e
Compare
0df336e
to
27b1ab7
Compare
27b1ab7
to
511744f
Compare
580c3a9
to
ebcdb2e
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.
LGTM
Merging with two approvals. Thanks all! |
...and get closer to absence of anonymous unions and raw pointers.