Skip to content

feat: add clang-tidy to CI#95

Merged
junrushao merged 2 commits into
apache:mainfrom
junrushao:2025-10-07/clang-tidy-in-ci
Oct 28, 2025
Merged

feat: add clang-tidy to CI#95
junrushao merged 2 commits into
apache:mainfrom
junrushao:2025-10-07/clang-tidy-in-ci

Conversation

@junrushao

@junrushao junrushao commented Oct 8, 2025

Copy link
Copy Markdown
Member

Depends on #197. This PR adds clang-tidy to per-commit tests on branch main to detect potential breaks.

@junrushao junrushao marked this pull request as ready for review October 8, 2025 03:05
@junrushao junrushao force-pushed the 2025-10-07/clang-tidy-in-ci branch 3 times, most recently from 7a65d3f to 85b106f Compare October 8, 2025 03:31
@junrushao junrushao force-pushed the 2025-10-07/clang-tidy-in-ci branch 3 times, most recently from 6f73b97 to 285a59c Compare October 27, 2025 17:46
@junrushao junrushao force-pushed the 2025-10-07/clang-tidy-in-ci branch from 4f51be7 to 741d35c Compare October 28, 2025 02:04
tqchen pushed a commit that referenced this pull request Oct 28, 2025
Following #78, this PR addresses
new warnings from clang-tidy.

After this PR being merged, PR #95 will run clang-tidy tests per-commit
to `main` branch to make sure it always passes.
@junrushao junrushao force-pushed the 2025-10-07/clang-tidy-in-ci branch from 741d35c to 2f2b6ef Compare October 28, 2025 17:19
@junrushao junrushao force-pushed the 2025-10-07/clang-tidy-in-ci branch from 2f2b6ef to cbe1fb8 Compare October 28, 2025 17:20
@junrushao junrushao merged commit 0899b5d into apache:main Oct 28, 2025
7 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