Use Boost::json in olp-cpp-sdk-authentication#1674
Use Boost::json in olp-cpp-sdk-authentication#1674rustam-gamidov-here merged 2 commits intofeature_boost_jsonfrom
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## feature_boost_json #1674 +/- ##
=====================================================
Coverage ? 80.38%
=====================================================
Files ? 351
Lines ? 14018
Branches ? 1518
=====================================================
Hits ? 11267
Misses ? 2134
Partials ? 617 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
a3a9336 to
80cc913
Compare
andrey-kashcheev
left a comment
There was a problem hiding this comment.
Do we need to have JSON sources to be included here in auth as well? It's currently in core, but does it work with visibility=hidden?
80cc913 to
e362140
Compare
Nice catch! Looks like it should be in all projects. Not sure how to verify on CI because in tests we are accessing not exposed classes. |
Migrating from RapidJSON Relates-To: OCMAM-443 Signed-off-by: Rustam Gamidov <ext-rustam.gamidov@here.com>
Migrating from RapidJSON Relates-To: OCMAM-443 Signed-off-by: Rustam Gamidov <ext-rustam.gamidov@here.com>
e362140 to
787a8ee
Compare
Migrating from RapidJSON
Relates-To: OCMAM-443