endee/.github/workflows/linting.yml at 4058e30e654a6167b20d608c23fc486e9616828b · endee-io/endee · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
# CONTINUOUS INTEGRATION WORKFLOW
# LINT TEST -> UNIT TEST -> BUILD


name: Continuous Integration

# ========== TRIGGER EVENT ==========
on:
  # PUSH EVENT
  # push:
  #   branches: [ feature/linting_tests ]
  # PULL REQUEST EVENT
  pull_request:
    branches: [ master]


# ========== JOBS ==========
jobs:

  # ----- JOB: 1 -> LINT AND CODE QUALITY
  lint:
    name: Lint and Code Quality Check
    runs-on: ubuntu-latest

    steps:

      #---------------- STEP 1: CHECKOUT TO SOURCE CODE
      - name: Checkout Source Code
        uses: actions/checkout@v4

      #---------------- STEP 2: CACHE FIRST, THEN INSTALL
      - name: Cache apt packages
        uses: awalsh128/cache-apt-pkgs-action@latest
        with:
          packages: clang-format clang-tidy shellcheck cmake libcurl4-openssl-dev
          version: 1.0

      # - name: Cache apt packages
      #   uses: awalsh128/cache-apt-pkgs-action@latest
      #   with:
      #     packages: clang-format clang-tidy shellcheck cmake libcurl4-openssl-dev
      #     version: 1.0

      # RUN CLANG-FORMAT FOR CODE STYLE, INDENTATIONS, SPACING
      # DRY RUN MAKE SURE IT WON'T CHANGE CODE ONLY IT WILL CHECK FORMATTING
      - name: C++ and HPP formatting
        run: |
          CHANGED=$(git diff --name-only origin/${{ github.base_ref }}...HEAD \
            | grep -E '\.(cpp|hpp)$' \
            | grep -v 'third_party/' \
            || true)

          if [ -z "$CHANGED" ]; then
            echo "No C++/HPP files changed — skipping"
            exit 0
          fi

          echo "Checking files:"
          echo "$CHANGED"
          echo "$CHANGED" | xargs clang-format --dry-run --Werror

      - name: C++ Static Analysis
        run: |
          CHANGED=$(git diff --name-only origin/${{ github.base_ref }}...HEAD \
            | grep -E '\.cpp$' \
            | grep -v 'third_party/' \
            || true)

          if [ -z "$CHANGED" ]; then
            echo "No .cpp files changed — skipping"
            exit 0
          fi

          rm -rf build && mkdir -p build
          cd build
          cmake .. \
            -DCMAKE_BUILD_TYPE=Release \
            -DUSE_AVX2=ON \
            -DCMAKE_EXPORT_COMPILE_COMMANDS=ON
          cd ..

          echo "Checking files:"
          echo "$CHANGED"
          echo "$CHANGED" | xargs clang-tidy \
            -p build/compile_commands.json \
            --header-filter='^.*(src)/.*'

      - name: Lint Shell Scripts
        run: |
          CHANGED=$(git diff --name-only origin/${{ github.base_ref }}...HEAD \
            | grep -E '\.sh$' \
            | grep -v 'build/' \
            | grep -v 'third_party/' \
            || true)

          if [ -z "$CHANGED" ]; then
            echo "No shell scripts changed — skipping"
            exit 0
          fi

          echo "Checking files:"
          echo "$CHANGED"
          echo "$CHANGED" | xargs shellcheck -x