-
Notifications
You must be signed in to change notification settings - Fork 30
Updates for fxamacker/cbor 2.9.0 release #218
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #218 +/- ##
==========================================
+ Coverage 91.23% 92.33% +1.10%
==========================================
Files 13 13
Lines 2133 1723 -410
==========================================
- Hits 1946 1591 -355
+ Misses 128 73 -55
Partials 59 59 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi, I see the latest release was a year ago, may it be possible to include this PR in a fresh one? |
|
I ran into this one too -- would be great to have this one merged, thank you! |
thomas-fossati
left a comment
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!
|
@gibmat, can you please resolve the conflicts? |
Signed-off-by: Mathias Gibbens <[email protected]>
53a220f to
93ffa3b
Compare
SteveLasker
left a comment
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, @gibmat
The format of some errors changed with the fxamacker/cbor 2.9.0 release.