cpp-linter · 2bndy5 · Dec 12, 2024 · Dec 12, 2024 · Dec 12, 2024 · Dec 12, 2024
diff --git a/.clang-format b/.clang-format
@@ -1,3 +1,3 @@
 ---
 Language:        Cpp
-BasedOnStyle:  WebKit
+BasedOnStyle:  WebKit
diff --git a/.clang-tidy b/.clang-tidy
@@ -183,3 +183,4 @@ CheckOptions:
     value:           '1'
   - key:             readability-uppercase-literal-suffix.NewSuffixes
     value:           ''
+...
diff --git a/.github/workflows/cpp-lint-action.yml b/.github/workflows/cpp-lint-action.yml
@@ -32,7 +32,7 @@ jobs:
         run: mkdir build && cmake -Bbuild src
 
       - name: Run linter as action
-        uses: cpp-linter/cpp-linter-action@latest
+        uses: cpp-linter/cpp-linter-action@main
         id: linter
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -42,17 +42,17 @@ jobs:
           # to ignore all build folder contents
           ignore: build
           database: build
-          verbosity: 9
+          verbosity: debug
           version: ${{ matrix.clang-version }}
-          thread-comments: ${{ matrix.clang-version == '12' }}
-          file-annotations: ${{ matrix.clang-version == '12' }}
+          thread-comments: ${{ matrix.clang-version == '16' }}
+          file-annotations: ${{ matrix.clang-version == '16' }}
           extra-args: -std=c++14 -Wall
 
       - name: Fail fast?!
         # if: steps.linter.outputs.checks-failed > 0
         run: | 
-          echo "some linter checks failed"
-          echo "${{ steps.linter.outputs.checks-failed }}"
-          echo "${{ env.checks-failed }}"
+          echo "checks-failed: ${{ steps.linter.outputs.checks-failed }}"
+          echo "clang-tidy-checks-failed: ${{ steps.linter.outputs.clang-tidy-checks-failed }}"
+          echo "clang-format-checks-failed: ${{ steps.linter.outputs.clang-format-checks-failed }}"
         # for actual deployment
         # run: exit 1
diff --git a/.github/workflows/cpp-lint-package.yml b/.github/workflows/cpp-lint-package.yml
@@ -5,8 +5,9 @@ on:
     inputs:
       branch:
         description: 'which branch to test'
-        default: 'main'
+        default: 'hide-reuse-pr-reviews'
         required: true
+  pull_request:
 
 jobs:
   cpp-linter:
@@ -16,7 +17,8 @@ jobs:
       matrix:
         clang-version: ['7', '8', '9','10', '11', '12', '13', '14', '15', '16', '17']
         repo: ['cpp-linter/cpp-linter']
-        branch: ['${{ inputs.branch }}']
+        branch:
+          - ${{ github.event_name == 'workflow_dispatch' && inputs.branch || 'hide-reuse-pr-reviews' }}
       fail-fast: false
 
     steps:
@@ -66,9 +68,14 @@ jobs:
           --extra-arg="-std=c++14 -Wall"
           --thread-comments=${{ matrix.clang-version == '17' && 'update' }}
           -a=${{ matrix.clang-version == '17' }}
+          --tidy-review=${{ matrix.clang-version == '17' }}
+          --format-review=${{ matrix.clang-version == '17' }}
 
       - name: Fail fast?!
         if: steps.linter.outputs.checks-failed > 0
-        run: echo "Some files failed the linting checks!"
+        run: |
+          echo "checks-failed: ${{ steps.linter.outputs.checks-failed }}"
+          echo "clang-tidy-checks-failed: ${{ steps.linter.outputs.clang-tidy-checks-failed }}"
+          echo "clang-format-checks-failed: ${{ steps.linter.outputs.clang-format-checks-failed }}"
         # for actual deployment
         # run: exit 1
diff --git a/src/demo.cpp b/src/demo.cpp
@@ -1,17 +1,18 @@
 /** This is a very ugly test code (doomed to fail linting) */
 #include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
+#include <stdio.h>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
-#include "demo.hpp"
-#include <cstdio>
-#include <cstddef>
-#include <stdio.h>
+#include <cstdio>
 
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-
+auto main() -> int
+{
-// using size_t from cstddef
-size_t dummyFunc(size_t i) { return i; }
 
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-
+int main()
+{
-int main()
-{
-    for (;;)
-        break;
+
+
+int main(){
+
+    for (;;) break;
+
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
-
-
-int main(){
-
-    for (;;) break;
+    for (;;)
+        break;
-
-
-int main(){
-
-    for (;;) break;
+    for (;;) {
+        break;
+    }
 
     printf("Hello world!\n");
 
-    return 0;
-}
+
+
+
+    return 0;}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
-
-
-
-    return 0;}
+    return 0;
+}
-    return 0;}
+    return 0;
+}
diff --git a/src/demo.hpp b/src/demo.hpp
@@ -1,16 +1,14 @@
 #pragma once



 class Dummy {
     char* useless;
     int numb;
+    Dummy() :numb(0), useless("\0"){}
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
-    char* useless;
-    int numb;
-    Dummy() :numb(0), useless("\0"){}
+    char* useless { "\0" };
+    int numb { 0 };
+    Dummy() { }
 
     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_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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
-    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; }
 };
 
 
@@ -28,14 +26,11 @@
 
 
 
-
-
-
-
 
 
 struct LongDiff
 {
+
     long diff;
 
 };