Skip to content

build: Use apple-clang 21; remove SourceLocation stub#3123

Merged
mathbunnyru merged 5 commits into
XRPLF:developfrom
mathbunnyru:up_cpp_apple_clang
Jun 30, 2026
Merged

build: Use apple-clang 21; remove SourceLocation stub#3123
mathbunnyru merged 5 commits into
XRPLF:developfrom
mathbunnyru:up_cpp_apple_clang

Conversation

@mathbunnyru

Copy link
Copy Markdown
Collaborator

No description provided.

@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.61%. Comparing base (c2b129b) to head (2f75d4f).

Files with missing lines Patch % Lines
src/util/log/Logger.cpp 83.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3123      +/-   ##
===========================================
- Coverage    82.64%   82.61%   -0.04%     
===========================================
  Files          397      397              
  Lines        16034    16034              
  Branches      8399     8399              
===========================================
- Hits         13252    13247       -5     
- Misses        1660     1667       +7     
+ Partials      1122     1120       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mathbunnyru mathbunnyru changed the title build: Use apple-clang 21 and cppstd 23 build: Use apple-clang 21; remove SourceLocation stub Jun 30, 2026

@godexsoft godexsoft left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay. hopefully it really does work on all platforms now :rage2:

@mathbunnyru mathbunnyru merged commit bc556fa into XRPLF:develop Jun 30, 2026
158 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants