-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add Deserializer
#767
Add Deserializer
#767
Conversation
|
d0446f5
to
e4f5d51
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #767 +/- ##
==========================================
- Coverage 40.52% 40.41% -0.11%
==========================================
Files 104 105 +1
Lines 6821 6839 +18
==========================================
Hits 2764 2764
- Misses 4057 4075 +18
☔ View full report in Codecov by Sentry. |
fd77cd4
to
47719eb
Compare
47719eb
to
55ca1e2
Compare
fda61ab
to
9dc85ef
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.
clang-tidy
found issue(s) with the introduced code (1/1)
a756bb4
to
6e28bc0
Compare
2c132f6
to
17c2e6a
Compare
17c2e6a
to
9fede47
Compare
9fede47
to
cc10a97
Compare
cc10a97
to
093801f
Compare
093801f
to
be1b8d9
Compare
Using this as testing grounds for the clang-tidy fix |
f42b80f
to
c2142b8
Compare
c2142b8
to
9aefb17
Compare
Description
Adds the
Deserializer
base class.Checklist
Ensure test coverage.(will be covered in tests of its implementations)