[clang-tools-extra] [llvm] [WIP] Add clang tidy premerge CI [WIP] (PR #154223)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 21 22:09:16 PDT 2025
github-actions[bot] wrote:
<!--LLVM CODE LINT COMMENT: clang-tidy-->
:warning: C/C++ code linter clang-tidy found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
```bash
git diff -U0 origin/main..HEAD -- clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp |
python3 clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py \
-path build -p1 -quiet
```
</details>
<details>
<summary>
View the output from clang-tidy here.
</summary>
```
clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp:22:19: warning: invalid case style for variable 'beforeArgumentsRange' [readability-identifier-naming]
22 | CharSourceRange beforeArgumentsRange = Lexer::makeFileCharRange(
| ^~~~~~~~~~~~~~~~~~~~
| BeforeArgumentsRange
23 | CharSourceRange::getCharRange(Call->getBeginLoc(), Arg->getBeginLoc()),
24 | SM, LangOpts);
25 | CharSourceRange AfterArgumentsRange = Lexer::makeFileCharRange(
26 | CharSourceRange::getCharRange(Call->getEndLoc(),
27 | Call->getEndLoc().getLocWithOffset(1)),
28 | SM, LangOpts);
29 |
30 | if (beforeArgumentsRange.isValid() && AfterArgumentsRange.isValid()) {
| ~~~~~~~~~~~~~~~~~~~~
| BeforeArgumentsRange
31 | Diag << FixItHint::CreateRemoval(beforeArgumentsRange)
| ~~~~~~~~~~~~~~~~~~~~
| BeforeArgumentsRange
```
</details>
https://github.com/llvm/llvm-project/pull/154223
More information about the cfe-commits
mailing list