From e4bcf16bb837f84a11333c054fa60167712e780f Mon Sep 17 00:00:00 2001 From: Brendan <2bndy5@gmail.com> Date: Mon, 5 Oct 2026 23:00:40 -0700 Subject: [PATCH 1/3] prep for PR --- .github/workflows/cpp-lint-package.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/cpp-lint-package.yml b/.github/workflows/cpp-lint-package.yml index 628c71a..87aed00 100644 --- a/.github/workflows/cpp-lint-package.yml +++ b/.github/workflows/cpp-lint-package.yml @@ -7,6 +7,7 @@ on: description: 'which branch to test' default: 'main' required: true + pull_request: jobs: pre-seed-versions: @@ -43,7 +44,7 @@ jobs: matrix: clang-version: ${{ fromJSON(needs.pre-seed-versions.outputs.versions) }} repo: ['cpp-linter/cpp-linter'] - branch: ['${{ inputs.branch }}'] + branch: ["${{ github.event_name == 'workflow_dispatch' && inputs.branch || 'git-bot-feedback' }}"] fail-fast: false steps: From 2c342a74d93278eab837efd8f188cf2893d88027 Mon Sep 17 00:00:00 2001 From: Brendan <2bndy5@gmail.com> Date: Mon, 5 Oct 2026 23:00:59 -0700 Subject: [PATCH 2/3] make src/demo changes --- src/demo.cpp | 21 +++++++++++---------- src/demo.hpp | 11 +++-------- 2 files changed, 14 insertions(+), 18 deletions(-) diff --git a/src/demo.cpp b/src/demo.cpp index 0c1db60..1bf553e 100644 --- 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 -#include +#include -// using size_t from cstddef -size_t dummyFunc(size_t i) { return i; } -int main() -{ - for (;;) - break; + + +int main(){ + + for (;;) break; + printf("Hello world!\n"); - return 0; -} + + + + return 0;} diff --git a/src/demo.hpp b/src/demo.hpp index 2695731..f93d012 100644 --- a/src/demo.hpp +++ b/src/demo.hpp @@ -5,12 +5,10 @@ class Dummy { char* useless; int numb; + Dummy() :numb(0), useless("\0"){} public: - void *not_usefull(char *str){ - useless = str; - return 0; - } + void *not_useful(char *str){useless = str;} }; @@ -28,14 +26,11 @@ class Dummy { - - - - struct LongDiff { + long diff; }; From 6b1fa3683f04d91da2cf861b2c26c301ec751022 Mon Sep 17 00:00:00 2001 From: Brendan <2bndy5@gmail.com> Date: Mon, 5 Oct 2026 23:17:20 -0700 Subject: [PATCH 3/3] enable step-summary and PR reviews --- .github/workflows/cpp-lint-package.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/cpp-lint-package.yml b/.github/workflows/cpp-lint-package.yml index 87aed00..28bb0ee 100644 --- a/.github/workflows/cpp-lint-package.yml +++ b/.github/workflows/cpp-lint-package.yml @@ -99,6 +99,9 @@ jobs: --extra-arg="-std=c++14 -Wall" --thread-comments=${{ matrix.clang-version == '17' && 'update' }} -a=${{ matrix.clang-version == '17' }} + --format-review ${{ matrix.clang-version == '17' }} + --tidy-review ${{ matrix.clang-version == '17' }} + --step-summary ${{ matrix.clang-version == '17' }} - name: Fail fast?! if: steps.linter.outputs.checks-failed > 0