Skip to content

test: e2e of cpp-linter-action@79dfaaef8e18815e923592c1d5098c7e0d25ecf6 (clang 21) - #74

Closed
github-actions[bot] wants to merge 1 commit into
masterfrom
e2e/run-43102458984
Closed

github-actions[bot] wants to merge 1 commit into
masterfrom
e2e/run-43102458984

Conversation

@github-actions

Copy link
Copy Markdown

Opened by the e2e on a real PR workflow. Do not merge. It is closed automatically when all checks pass.

@github-actions

Copy link
Copy Markdown
Author

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
clang-tidy (v21.1.8) reports: 5 concern(s)
  • src/demo.cpp:9:5: warning: [misc-use-internal-linkage]

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

        9 | int scale(int value)
          |     ^
          | static 
  • src/demo.cpp:9:5: warning: [modernize-use-trailing-return-type]

    use a trailing return type for this function

        9 | int scale(int value)
          | ~~~ ^               
          | auto                 -> int
  • src/demo.cpp:11:19: warning: [readability-braces-around-statements]

    statement should be inside braces

       11 |     if (value > 0)
          |                   ^
          |                    {
       12 |         return value*3;
          |                        
  • src/demo.cpp:21:5: warning: [cppcoreguidelines-pro-type-vararg]

    do not call c-style vararg functions

       21 |     printf("Hello world! %d\n", scale( 14 ));
          |     ^
  • src/demo.cpp:21:40: warning: [cppcoreguidelines-avoid-magic-numbers]

    14 is a magic number; consider replacing it with a named constant

       21 |     printf("Hello world! %d\n", scale( 14 ));
          |                                        ^

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

@github-actions

Copy link
Copy Markdown
Author

e2e result: failed

cpp-linter-action 79dfaaef8e18815e923592c1d5098c7e0d25ecf6 (79dfaae), clang 21,
workflow run

Check Actual Expected
lint jobs cancelled success ❌
outputs on ubuntu-latest (total/tidy/format) 6/5/1 tidy > 0, format > 0, total = tidy + format ✅
outputs on macos-latest (total/tidy/format) none/none/none tidy > 0, format > 0, total = tidy + format ❌
outputs on windows-latest (total/tidy/format) 6/5/1 tidy > 0, format > 0, total = tidy + format ✅
thread comments 1 1 ✅
PR reviews 0 (latest: none) 1 or more (latest: CHANGES_REQUESTED) ❌
review comments 0 1 or more ❌
file annotations on src/ 6 2 or more ✅

@github-actions
github-actions Bot deleted the e2e/run-43102458984 branch November 11, 2026 22:35
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