Test py git bot feedback by 2bndy5 · Pull Request #63 · cpp-linter/test-cpp-linter-action

2bndy5 · 2026-10-06T06:02:05Z

Summary

testing cpp-linter/cpp-linter#226 in PR context

github-actions · 2026-10-06T06:03:14Z

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format (v17.0.6) reports: 2 file(s) not formatted

src/demo.cpp
src/demo.hpp

clang-tidy (v17.0.6) reports: 8 concern(s)

src/demo.cpp:2:1: warning: [misc-include-cleaner]

included header demo.hpp is not used directly

    2 | #include "demo.hpp"
      | ^~~~~~~~~~~~~~~~~~~
    3 | #include <stdio.h>

src/demo.cpp:3:10: warning: [modernize-deprecated-headers]

inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
```
    3 | #include <stdio.h>
      |          ^~~~~~~~~
      |          <cstdio>
```
src/demo.cpp:8:5: warning: [modernize-use-trailing-return-type]

use a trailing return type for this function
```
    8 | int main(){
      | ~~~ ^     
      | auto       -> int
```
src/demo.cpp:10:13: warning: [readability-braces-around-statements]

statement should be inside braces
```
   10 |     for (;;) break;
      |             ^      
      |              {
```
src/demo.cpp:13:5: warning: [cppcoreguidelines-pro-type-vararg]

do not call c-style vararg functions
```
   13 |     printf("Hello world!\n");
      |     ^
```

src/demo.hpp:6:11: warning: [cppcoreguidelines-use-default-member-init]

use default member initializer for 'useless'

    6 |     char* useless;
      |           ^      
      |                  {"\0"}
    7 |     int numb;
    8 |     Dummy() :numb(0), useless("\0"){}
      |                       ~~~~~~~~~~~~~

src/demo.hpp:7:9: warning: [cppcoreguidelines-use-default-member-init]

use default member initializer for 'numb'

    7 |     int numb;
      |         ^   
      |             {0}
    8 |     Dummy() :numb(0), useless("\0"){}
      |              ~~~~~~~

src/demo.hpp:11:11: warning: [modernize-use-trailing-return-type]

use a trailing return type for this function

   11 |     void *not_useful(char *str){useless = str;}
      |     ~~~~~~^                    
      |     auto                        -> void *

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

github-actions

Cpp-linter Review

Used clang-tidy v17.0.6

Click here for the full clang-tidy patch

diff --git a/src/demo.cpp b/src/demo.cpp
index 1bf553e..8caf462 100644
--- a/src/demo.cpp
+++ b/src/demo.cpp
@@ -2,2 +2 @@
-#include "demo.hpp"
-#include <stdio.h>
+#include <cstdio>
@@ -4,0 +4,2 @@
+auto main() -> int
+{
@@ -6,6 +7,3 @@
-
-
-int main(){
-
-    for (;;) break;
-
+    for (;;) {
+        break;
+    }
@@ -18 +16,2 @@ int main(){
-    return 0;}
+    return 0;
+}
diff --git a/src/demo.hpp b/src/demo.hpp
index f93d012..1cf7f24 100644
--- a/src/demo.hpp
+++ b/src/demo.hpp
@@ -6,3 +6,3 @@ class Dummy {
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
@@ -10,2 +10,2 @@ class Dummy {
-    public:
-    void *not_useful(char *str){useless = str;}
+public:
+    auto not_useful(char* str) -> void* { useless = str; }

Used clang-format v17.0.6

Only 5 out of 7 clang-format concerns fit within this pull request's diff.

Click here for the full clang-format patch

diff --git a/src/demo.cpp b/src/demo.cpp
index 1bf553e..86d9f03 100644
--- a/src/demo.cpp
+++ b/src/demo.cpp
@@ -4,0 +5,2 @@
+int main()
+{
@@ -6,6 +8,2 @@
-
-
-int main(){
-
-    for (;;) break;
-
+    for (;;)
+        break;
@@ -15,4 +13,2 @@ int main(){
-
-
-
-    return 0;}
+    return 0;
+}
diff --git a/src/demo.hpp b/src/demo.hpp
index f93d012..dad4fd6 100644
--- a/src/demo.hpp
+++ b/src/demo.hpp
@@ -3,2 +2,0 @@
-
-
@@ -8,4 +6,8 @@ class Dummy {
-    Dummy() :numb(0), useless("\0"){}
-
-    public:
-    void *not_useful(char *str){useless = str;}
+    Dummy()
+        : numb(0)
+        , useless("\0")
+    {
+    }
+
+public:
+    void* not_useful(char* str) { useless = str; }
@@ -14,19 +16 @@ class Dummy {
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-struct LongDiff
-{
+struct LongDiff {
@@ -35 +18,0 @@ struct LongDiff
-

Have any feedback or feature suggestions? Share it here.

github-actions · 2026-10-06T06:18:31Z


-// using size_t from cstddef
-size_t dummyFunc(size_t i) { return i; }



clang-format suggestion

Suggested change

int main()

{

github-actions · 2026-10-06T06:18:31Z

+
+
+int main(){
+
+    for (;;) break;
+


clang-format suggestion

Suggested change

int main(){

for (;;) break;

for (;;)

break;

clang-tidy diagnostics

use a trailing return type for this function [modernize-use-trailing-return-type]

statement should be inside braces [readability-braces-around-statements]

Suggested change

int main(){

for (;;) break;

for (;;) {

break;

}

github-actions · 2026-10-06T06:18:31Z

+
+
+
+    return 0;}


clang-format suggestion

Suggested change

return 0;}

return 0;

}

github-actions · 2026-10-06T06:18:31Z

+    Dummy() :numb(0), useless("\0"){}

    public:
-    void *not_usefull(char *str){
-        useless = str;
-        return 0;
-    }
+    void *not_useful(char *str){useless = str;}


clang-format suggestion

Suggested change

Dummy() :numb(0), useless("\0"){}

public:

void *not_usefull(char *str){

useless = str;

return 0;

}

void *not_useful(char *str){useless = str;}

Dummy()

: numb(0)

, useless("\0")

{

}

public:

void* not_useful(char* str) { useless = str; }

github-actions · 2026-10-06T06:18:31Z

 {
+
    long diff;



clang-format suggestion

Please remove the line(s)

35

github-actions · 2026-10-06T06:18:31Z

-#include <cstdio>
-#include <cstddef>
+#include <stdio.h>



clang-tidy suggestion

Suggested change

auto main() -> int

{

github-actions · 2026-10-06T06:18:31Z

+
+
+
+    return 0;}


clang-tidy suggestion

Suggested change

return 0;}

return 0;

}

github-actions · 2026-10-06T06:18:31Z

+    for (;;) break;
+

    printf("Hello world!\n");


clang-tidy diagnostic

src/demo.cpp:13:5: warning: [cppcoreguidelines-pro-type-vararg]

do not call c-style vararg functions

13 | printf("Hello world!\n"); | ^

github-actions · 2026-10-06T06:18:32Z

    char* useless;
    int numb;
+    Dummy() :numb(0), useless("\0"){}


clang-tidy diagnostics

use default member initializer for 'useless' [cppcoreguidelines-use-default-member-init]

use default member initializer for 'numb' [cppcoreguidelines-use-default-member-init]

Suggested change

char* useless;

int numb;

Dummy() :numb(0), useless("\0"){}

char* useless { "\0" };

int numb { 0 };

Dummy() { }

github-actions · 2026-10-06T06:18:32Z

    public:
-    void *not_usefull(char *str){
-        useless = str;
-        return 0;
-    }
+    void *not_useful(char *str){useless = str;}


clang-tidy diagnostics

use a trailing return type for this function [modernize-use-trailing-return-type]

Suggested change

public:

void *not_usefull(char *str){

useless = str;

return 0;

}

void *not_useful(char *str){useless = str;}

public:

auto not_useful(char* str) -> void* { useless = str; }

outdated review

github-actions

Cpp-linter Review

Used clang-tidy v17.0.6

Click here for the full clang-tidy patch

diff --git a/src/demo.cpp b/src/demo.cpp
index 1bf553e..8caf462 100644
--- a/src/demo.cpp
+++ b/src/demo.cpp
@@ -2,2 +2 @@
-#include "demo.hpp"
-#include <stdio.h>
+#include <cstdio>
@@ -4,0 +4,2 @@
+auto main() -> int
+{
@@ -6,6 +7,3 @@
-
-
-int main(){
-
-    for (;;) break;
-
+    for (;;) {
+        break;
+    }
@@ -18 +16,2 @@ int main(){
-    return 0;}
+    return 0;
+}
diff --git a/src/demo.hpp b/src/demo.hpp
index f93d012..1cf7f24 100644
--- a/src/demo.hpp
+++ b/src/demo.hpp
@@ -6,3 +6,3 @@ class Dummy {
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
@@ -10,2 +10,2 @@ class Dummy {
-    public:
-    void *not_useful(char *str){useless = str;}
+public:
+    auto not_useful(char* str) -> void* { useless = str; }

Used clang-format v17.0.6

Only 5 out of 7 clang-format concerns fit within this pull request's diff.

Click here for the full clang-format patch

diff --git a/src/demo.cpp b/src/demo.cpp
index 1bf553e..86d9f03 100644
--- a/src/demo.cpp
+++ b/src/demo.cpp
@@ -4,0 +5,2 @@
+int main()
+{
@@ -6,6 +8,2 @@
-
-
-int main(){
-
-    for (;;) break;
-
+    for (;;)
+        break;
@@ -15,4 +13,2 @@ int main(){
-
-
-
-    return 0;}
+    return 0;
+}
diff --git a/src/demo.hpp b/src/demo.hpp
index f93d012..dad4fd6 100644
--- a/src/demo.hpp
+++ b/src/demo.hpp
@@ -3,2 +2,0 @@
-
-
@@ -8,4 +6,8 @@ class Dummy {
-    Dummy() :numb(0), useless("\0"){}
-
-    public:
-    void *not_useful(char *str){useless = str;}
+    Dummy()
+        : numb(0)
+        , useless("\0")
+    {
+    }
+
+public:
+    void* not_useful(char* str) { useless = str; }
@@ -14,19 +16 @@ class Dummy {
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-struct LongDiff
-{
+struct LongDiff {
@@ -35 +18,0 @@ struct LongDiff
-

11 review comment(s) from a previous review were reused (not posted again).

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

2bndy5 added 2 commits October 5, 2026 23:00

prep for PR

e4bcf16

make src/demo changes

2c342a7

github-actions Bot requested changes Oct 6, 2026

View reviewed changes

2bndy5 mentioned this pull request Oct 6, 2026

feat: migrate to git-bot-feedback cpp-linter/cpp-linter#226

Draft

enable step-summary and PR reviews

6b1fa36

2bndy5 force-pushed the test-py-git-bot-feedback branch from 1711f3a to 6b1fa36 Compare October 6, 2026 15:59

This comment was marked as outdated.

Sign in to view

github-actions Bot requested changes Oct 8, 2026

View reviewed changes


		// using size_t from cstddef
		size_t dummyFunc(size_t i) { return i; }

Conversation

2bndy5 commented Oct 6, 2026

Summary

Uh oh!

github-actions Bot commented Oct 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Cpp-Linter Report ⚠️

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Cpp-linter Review

Used clang-tidy v17.0.6

Used clang-format v17.0.6

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-format suggestion

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-format suggestion

clang-tidy diagnostics

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-format suggestion

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-format suggestion

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-format suggestion

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-tidy suggestion

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-tidy suggestion

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-tidy diagnostic

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-tidy diagnostics

Uh oh!

github-actions Bot Oct 6, 2026

Choose a reason for hiding this comment

clang-tidy diagnostics

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Cpp-linter Review

Used clang-tidy v17.0.6

Used clang-format v17.0.6

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

github-actions Bot commented Oct 6, 2026 •

edited

Loading