-
Notifications
You must be signed in to change notification settings - Fork 34
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 Serializer
#652
Add Serializer
#652
Conversation
|
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)
0976f8c
to
1d995ee
Compare
No clang-tidy warnings found so I assume my comments were addressed
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #652 +/- ##
==========================================
- Coverage 36.52% 36.44% -0.08%
==========================================
Files 89 90 +1
Lines 6377 6390 +13
==========================================
Hits 2329 2329
- Misses 4048 4061 +13
☔ View full report in Codecov by Sentry. |
Co-authored-by: Luís Fonseca <[email protected]>
Closes #567.
Description
Adds the
Serializer
base class.Checklist
Ensure test coverage.Planning on leaving it to the actual implementations.