[shenxianpeng]

Temporary test PR, do not merge. Runs cpp-linter-action v3.0.0-rc0 (34d5522, cpp-linter py rc24 = rc.25) end to end: PR review (and the tidy-review alias), thread comment, step summary, summary output file, auto-fix, empty path inputs, and Linux/macOS/Windows vs v2. Results go to cpp-linter/cpp-linter-action#283.

[github-actions]

Cpp-linter Review

Used clang-format v21.1.8

Used clang-tidy v21.1.8

---

_{✨ Powered by cpp-linter -- ⭐ Star • ❤️ Sponsor • 💬 Feedback}

[github-actions]

clang-tidy diagnostic(s)

• function 'addedForV3Test' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage]
• use a trailing return type for this function [modernize-use-trailing-return-type]

Suggested change

	int addedForV3Test(int x){return x*2;}
	static auto addedForV3Test(int x) -> int { return x * 2; }

clang-tidy diagnostic

src/demo.cpp:19:25 warning: [readability-identifier-length]

parameter name 'x' is too short, expected at least 3 characters

   19 | int  addedForV3Test(int x){return x*2;}
      |                         ^

[github-actions]

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format (v21.1.8) reports: 1 file(s) not formatted

• src/demo.cpp (1 line)

clang-tidy (v21.1.8) reports: 3 concern(s)

• src/demo.cpp

  src/demo.cpp:19:6: warning: [misc-use-internal-linkage]
  ⚡ auto-fix included in generated patch¹

  function 'addedForV3Test' can be made static or moved into an anonymous namespace to enforce internal linkage

     19 | int  addedForV3Test(int x){return x*2;}
        |      ^
        | static 

• src/demo.cpp

  src/demo.cpp:19:6: warning: [modernize-use-trailing-return-type]
  ⚡ auto-fix included in generated patch¹

  use a trailing return type for this function

     19 | int  addedForV3Test(int x){return x*2;}
        | ~~~  ^                    
        | auto                       -> int

• src/demo.cpp

  src/demo.cpp:19:25: warning: [readability-identifier-length]

  parameter name 'x' is too short, expected at least 3 characters

     19 | int  addedForV3Test(int x){return x*2;}
        |                         ^

---

_{✨ Powered by cpp-linter -- ⭐ Star • ❤️ Sponsor • 💬 Feedback}

Footnotes

1. Cpp-linter generates a patch with fixes from both clang-tidy and clang-format. In the CI workflow, the patch can be found using cpp-linter's output fix-patch-path. ↩ ↩²