Skip to content

test: e2e cpp-linter-action v3.0.0-rc0 - #67

Closed
shenxianpeng wants to merge 3 commits into
masterfrom
claude/project-thread-3y9j3l
Closed

shenxianpeng wants to merge 3 commits into
masterfrom
claude/project-thread-3y9j3l

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Nov 1, 2026 •

Copy link
Copy Markdown
Member

Temporary test PR, will be closed without merging. Runs cpp-linter/[email protected] (cpp-linter v2.0.0rc23 from TestPyPI) on pull_request: deprecated tidy-review alias, pr-review + thread comment, then auto-fix. The push run on the same branch compares v3 and v2 across Linux x64/arm64, macOS, Windows x64/arm64 and clang 12/17/21/23.

github-actions[bot]

This comment was marked as outdated.

@github-actions

github-actions Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v21.1.6) reports: 2 concern(s)
  • src/demo.cpp

    src/demo.cpp:19:13: warning: [misc-use-anonymous-namespace]

    function 'addedForV3Test' declared 'static', move to anonymous namespace instead

       19 | static auto addedForV3Test(int x) -> int { return x * 2; }
          |             ^
  • src/demo.cpp

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

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

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

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

@github-actions
github-actions Bot dismissed their stale review November 1, 2026 21:18

outdated review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cpp-linter Review

Used clang-format v21.1.8

Used clang-tidy v21.1.6


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

Comment thread src/demo.cpp
return 0;
}

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

clang-tidy diagnostic

src/demo.cpp:19:13 warning: [misc-use-anonymous-namespace]

function 'addedForV3Test' declared 'static', move to anonymous namespace instead

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

clang-tidy diagnostic

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

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

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

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.

1 participant