You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by sz...@apache.org on 2022/08/25 15:35:47 UTC

[nifi-minifi-cpp] 01/03: MINIFICPP-1915 Fix clang-tidy for newly created branch

This is an automated email from the ASF dual-hosted git repository.

szaszm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/nifi-minifi-cpp.git

commit 462a7bc9c2e0448b47b0ab0afe477a14e198f1bf
Author: Gabor Gyimesi <ga...@gmail.com>
AuthorDate: Thu Aug 25 16:54:25 2022 +0200

    MINIFICPP-1915 Fix clang-tidy for newly created branch
    
    Closes #1398
    Signed-off-by: Marton Szasz <sz...@apache.org>
---
 .github/workflows/ci.yml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index ffd79921f..ff1350d8b 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -155,13 +155,21 @@ jobs:
         run: cd build && make flake8
       - id: files
         uses: Ana06/get-changed-files@v2.1.0
+        continue-on-error: true
       - name: clang-tidy
         run: |
+          if [[ -n "${{ steps.files.outputs.all }}" ]]; then
+            FILES="${{ steps.files.outputs.all }}"
+          else
+            git fetch origin main && git checkout -b main origin/main || true
+            FILES=$(git diff --name-only main ${{ github.sha }} --)
+            git checkout ${{ github.sha }}
+          fi
           # https://stackoverflow.com/questions/58466701/clang-tidy-cant-locate-stdlib-headers
           sed -i -e 's/\/usr\/lib\/ccache\/clang++-14/\/lib\/llvm-14\/bin\/clang++/g' build/compile_commands.json
           sed -i -e 's/\/usr\/lib\/ccache\/clang-14/\/lib\/llvm-14\/bin\/clang/g' build/compile_commands.json
 
-          ./run_clang_tidy.sh "${{ steps.files.outputs.all }}"
+          ./run_clang_tidy.sh "${FILES}"
   centos:
     name: "centos"
     runs-on: ubuntu-20.04