Test rust 3 by 2bndy5 · Pull Request #47 · cpp-linter/test-cpp-linter-action

2bndy5 · 2025-05-22T00:04:02Z

dismissing outdated reviews while keeping reused suggestions

github-actions · 2025-05-22T00:20:32Z

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

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

src/demo.cpp
src/demo.hpp

clang-tidy (v17.0.4) reports: 6 concern(s)

src/demo.cpp

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

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

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

src/demo.cpp:10:13: warning: [readability-braces-around-statements]

statement should be inside braces
```
   10 |     for (;;) break;
      |             ^      
      |              {
```
src/demo.cpp

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

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 *

Have any feedback or feature suggestions? Share it here.

github-actions

Cpp-linter Review

Used clang-format v17.0.4

Only 5 out of 7 new 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
-

Used clang-tidy v17.0.4

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; }

Have any feedback or feature suggestions? Share it here.

github-actions · 2025-05-22T00:20:33Z

src/demo.cpp

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


clang-format suggestions

Suggested change

int main(){

for (;;) break;

for (;;)

break;

clang-tidy diagnostic(s)

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 · 2025-05-22T00:20:34Z

src/demo.hpp

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

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


clang-format suggestions

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 · 2025-05-22T00:20:34Z

src/demo.hpp

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


clang-tidy diagnostic(s)

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; }

github-actions

Cpp-linter Review

Used clang-format v17.0.4

Only 5 out of 7 new 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
-

Used clang-tidy v17.0.4

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; }

Have any feedback or feature suggestions? Share it here.

github-actions · 2025-05-22T01:41:30Z

src/demo.hpp

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


clang-tidy suggestion

Suggested change

char* useless;

int numb;

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

char* useless { "\0" };

int numb { 0 };

Dummy() { }

github-actions

Cpp-linter Review

Used clang-format v17.0.4

Only 5 out of 7 new 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
-

Used clang-tidy v17.0.4

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; }

Have any feedback or feature suggestions? Share it here.

github-actions · 2025-05-22T01:58:50Z

src/demo.cpp

+
+
+
+    return 0;}


clang-format suggestions

Suggested change

return 0;}

return 0;

}

github-actions · 2025-05-22T01:58:50Z

src/demo.cpp

 #include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
+#include <stdio.h>


clang-tidy diagnostic(s)

included header demo.hpp is not used directly [misc-include-cleaner]

inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead [modernize-deprecated-headers]

Suggested change

#include "demo.hpp"

#include <cstdio>

#include <cstddef>

#include <stdio.h>

#include <cstdio>

outdated suggestion

update CI rm subdir param to pip install

github-actions

Cpp-linter Review

Used clang-format v17.0.4

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
-

Used clang-tidy v17.0.4

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; }

Have any feedback or feature suggestions? Share it here.

github-actions · 2025-08-18T10:54:54Z

src/demo.cpp


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



clang-format suggestions

Suggested change

int main()

{

github-actions · 2025-08-18T10:54:54Z

src/demo.cpp

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



clang-tidy suggestion

Suggested change

auto main() -> int

{

github-actions · 2025-08-18T10:54:54Z

src/demo.cpp

+
+
+
+    return 0;}


clang-tidy suggestion

Suggested change

return 0;}

return 0;

}

github-actions · 2025-08-18T10:54:55Z

src/demo.cpp

+    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 · 2025-08-18T10:54:55Z

src/demo.hpp

 {
+
    long diff;



clang-format suggestions

Please remove the line(s)

35

github-actions bot previously requested changes May 22, 2025

View reviewed changes

This comment was marked as outdated.

Sign in to view

2bndy5 added 7 commits August 18, 2025 03:47

prepare for rust pkg tests

359c499

enable PR reviews

5244d1b

switch to main branch

ebf8697

enable color logs

c54186b

update src/demo.* and clang configs

ab83618

turn off line filter

a44db76

bump to v2.0.0rc12

37b2c4e

update CI rm subdir param to pip install

2bndy5 force-pushed the test-rust branch from 6efa443 to 37b2c4e Compare August 18, 2025 10:50

github-actions bot requested changes Aug 18, 2025

View reviewed changes


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

Conversation

2bndy5 commented May 22, 2025

Uh oh!

github-actions bot commented May 22, 2025

Cpp-Linter Report ⚠️

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Cpp-linter Review

Used clang-format v17.0.4

Used clang-tidy v17.0.4

Uh oh!

github-actions bot May 22, 2025

Choose a reason for hiding this comment

clang-format suggestions

clang-tidy diagnostic(s)

Uh oh!

github-actions bot May 22, 2025

Choose a reason for hiding this comment

clang-format suggestions

Uh oh!

github-actions bot May 22, 2025

Choose a reason for hiding this comment

clang-tidy diagnostic(s)

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Cpp-linter Review

Used clang-format v17.0.4

Used clang-tidy v17.0.4

Uh oh!

github-actions bot May 22, 2025

Choose a reason for hiding this comment

clang-tidy suggestion

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Cpp-linter Review

Used clang-format v17.0.4

Used clang-tidy v17.0.4

Uh oh!

github-actions bot May 22, 2025

Choose a reason for hiding this comment

clang-format suggestions

Uh oh!

github-actions bot May 22, 2025

Choose a reason for hiding this comment

clang-tidy diagnostic(s)

Uh oh!

This comment was marked as outdated.

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-format v17.0.4

Used clang-tidy v17.0.4

Uh oh!

github-actions bot Aug 18, 2025

Choose a reason for hiding this comment

clang-format suggestions

Uh oh!

github-actions bot Aug 18, 2025

Choose a reason for hiding this comment

clang-tidy suggestion

Uh oh!

github-actions bot Aug 18, 2025

Choose a reason for hiding this comment

clang-tidy suggestion

Uh oh!

github-actions bot Aug 18, 2025

Choose a reason for hiding this comment

clang-tidy diagnostic

Uh oh!

github-actions bot Aug 18, 2025

Choose a reason for hiding this comment

clang-format suggestions