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
1.05.04 json encoder fixes v2 #2539
base: master
Are you sure you want to change the base?
Conversation
tagged as release
Randy Armstrong seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2539 +/- ##
===========================================
- Coverage 54.62% 44.16% -10.46%
===========================================
Files 342 296 -46
Lines 65082 61172 -3910
Branches 13350 12643 -707
===========================================
- Hits 35553 27019 -8534
- Misses 25661 31139 +5478
+ Partials 3868 3014 -854 ☔ View full report in Codecov by Sentry. |
Proposed changes
Fixes to the JSON encoder to support the RC + NodeId/ExpandedNodeId/QualifiedName encoding,