XRPLF / clio

An XRP Ledger API Server
https://xrpl.org
ISC License
61 stars 52 forks source link

Support clang 16 in docker CI #1348

Closed godexsoft closed 5 months ago

godexsoft commented 5 months ago

Fixes #1175

Currently clang-tidy is producing a lot of interesting output. But i think this can be handled outside of this PR now.

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 50.00000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 64.44%. Comparing base (1fe42c8) to head (1fd2018).

:exclamation: Current head 1fd2018 differs from pull request most recent head d6ce1f9. Consider uploading reports for the commit d6ce1f9 to get more accurate results

Files Patch % Lines
src/etl/impl/LedgerLoader.hpp 0.00% 2 Missing :warning:
src/etl/impl/SubscriptionSource.cpp 66.66% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1348 +/- ## =========================================== + Coverage 64.38% 64.44% +0.05% =========================================== Files 248 248 Lines 10544 10544 Branches 5839 5839 =========================================== + Hits 6789 6795 +6 + Misses 1920 1914 -6 Partials 1835 1835 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.