From b6cea20c55f663bb838c03b145a8c9ce5e2d83b5 Mon Sep 17 00:00:00 2001 From: Xianpeng Shen Date: Tue, 2 Sep 2025 22:44:57 +0300 Subject: [PATCH 1/2] Update thread-comments condition in self-test.yml Right now merge PR will be added thread-comments, see https://github.com/cpp-linter/cpp-linter-action/commit/b20a30fb194b1f5ef1469c8085a6e4d5b93f7443 I would like to disable it for merged commits. --- .github/workflows/self-test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/self-test.yml b/.github/workflows/self-test.yml index f8c1078..2f199fe 100644 --- a/.github/workflows/self-test.yml +++ b/.github/workflows/self-test.yml @@ -62,7 +62,7 @@ jobs: database: build verbosity: debug version: ${{ matrix.clang-version }} - thread-comments: ${{ matrix.clang-version == '12' && 'update' }} + thread-comments: ${{ github.event_name == 'pull_request' && matrix.clang-version == '12' && 'update' }} file-annotations: ${{ runner.os == 'Linux' && matrix.clang-version == '12' }} step-summary: ${{ matrix.clang-version == '12' }} extra-args: -std=c++14 -Wall From a6c51541296051dc05a001b1c92c3acad329d568 Mon Sep 17 00:00:00 2001 From: Xianpeng Shen Date: Wed, 3 Sep 2025 05:00:57 +0300 Subject: [PATCH 2/2] Change clang version to 16 in self-test workflow --- .github/workflows/self-test.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/self-test.yml b/.github/workflows/self-test.yml index 2f199fe..bbcf594 100644 --- a/.github/workflows/self-test.yml +++ b/.github/workflows/self-test.yml @@ -62,9 +62,9 @@ jobs: database: build verbosity: debug version: ${{ matrix.clang-version }} - thread-comments: ${{ github.event_name == 'pull_request' && matrix.clang-version == '12' && 'update' }} - file-annotations: ${{ runner.os == 'Linux' && matrix.clang-version == '12' }} - step-summary: ${{ matrix.clang-version == '12' }} + thread-comments: ${{ github.event_name == 'pull_request' && matrix.clang-version == '16' && 'update' }} + file-annotations: ${{ runner.os == 'Linux' && matrix.clang-version == '16' }} + step-summary: ${{ matrix.clang-version == '16' }} extra-args: -std=c++14 -Wall - name: Fail fast?!