Skip to content

chore: Update .clang-tidy to check all C++ source files#6711

Merged
myrrc merged 1 commit intovortex-data:developfrom
SYaoJun:228_tidy
Feb 27, 2026
Merged

chore: Update .clang-tidy to check all C++ source files#6711
myrrc merged 1 commit intovortex-data:developfrom
SYaoJun:228_tidy

Conversation

@SYaoJun
Copy link
Contributor

@SYaoJun SYaoJun commented Feb 27, 2026

Summary

Closes: #000

Testing

Signed-off-by: syaojun <libevent@yeah.net>
@SYaoJun
Copy link
Contributor Author

SYaoJun commented Feb 27, 2026

vortex-cxx
├── CMakeLists.txt
├── Cargo.toml
├── README.md
├── cpp
├── examples
└── src

src directory is all rust files, so it is unneccessary to include.

@myrrc myrrc self-assigned this Feb 27, 2026
@myrrc myrrc added the changelog/chore A trivial change label Feb 27, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 27, 2026

Merging this PR will improve performance by 17.64%

⚡ 1 improved benchmark
✅ 953 untouched benchmarks
⏩ 1466 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation chunked_opt_bool_canonical_into[(10, 1000)] 1.6 ms 1.4 ms +17.64%

Comparing SYaoJun:228_tidy (79bba54) with develop (fc3af37)

Open in CodSpeed

Footnotes

  1. 1466 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@myrrc myrrc self-requested a review February 27, 2026 16:38
Copy link
Contributor

@myrrc myrrc left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@myrrc myrrc merged commit 1786200 into vortex-data:develop Feb 27, 2026
57 of 58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants