Skip to content

feat: remove duplicate .clang-format and move it to top layer#850

Merged
yangxk1 merged 2 commits intoapache:mainfrom
SYaoJun:duplic_clang_format
Feb 24, 2026
Merged

feat: remove duplicate .clang-format and move it to top layer#850
yangxk1 merged 2 commits intoapache:mainfrom
SYaoJun:duplic_clang_format

Conversation

@SYaoJun
Copy link
Contributor

@SYaoJun SYaoJun commented Feb 9, 2026

Reason for this PR

There are two .clang-format under different directories.

We could remove one of them and move it to top directory, then other bindings can share the same configuration.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@codecov-commenter
Copy link

codecov-commenter commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.21%. Comparing base (02bd3fe) to head (16291cf).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #850      +/-   ##
============================================
- Coverage     80.21%   80.21%   -0.01%     
  Complexity      615      615              
============================================
  Files            93       93              
  Lines         10257    10258       +1     
  Branches       1049     1050       +1     
============================================
  Hits           8228     8228              
- Misses         1789     1790       +1     
  Partials        240      240              
Flag Coverage Δ
cpp 71.50% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

@SYaoJun SYaoJun force-pushed the duplic_clang_format branch 3 times, most recently from 881f038 to 0c32747 Compare February 12, 2026 12:01
@SYaoJun SYaoJun closed this Feb 12, 2026
@SYaoJun SYaoJun reopened this Feb 18, 2026
@SYaoJun SYaoJun force-pushed the duplic_clang_format branch from 0c32747 to aef79e7 Compare February 18, 2026 16:02
Copy link
Contributor

@yangxk1 yangxk1 left a comment

Choose a reason for hiding this comment

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

LGTM~

@yangxk1 yangxk1 merged commit 6e72a88 into apache:main Feb 24, 2026
13 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.

3 participants