Skip to content

Commit b28b4f2

Browse files
[SQUASH ME] Fixed Missing Pkgs
1 parent f1df2c6 commit b28b4f2

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

.github/workflows/test.yml

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,9 @@ jobs:
122122
fail-fast: false
123123
matrix:
124124
include:
125-
- { name: 'C/C++', script: 'check-format.sh' }
126-
- { name: 'Python', script: 'check-format-py.sh' }
127-
- { name: 'Python Lint', script: 'pylint_check.py' }
125+
- { name: 'C/C++', script: 'check-format.sh' , pkgs: 'clang-format-18' }
126+
- { name: 'Python', script: 'check-format-py.sh', pkgs: '' }
127+
- { name: 'Python Lint', script: 'pylint_check.py' , pkgs: '' }
128128
name: 'F: ${{ matrix.name }}'
129129
steps:
130130

@@ -135,11 +135,17 @@ jobs:
135135
python-version: 3.10.10
136136

137137
- uses: actions/checkout@v4
138-
with:
139-
submodules: 'true'
138+
# NOTE: We do not need sub-modules. We do not check sub-modules for formatting.
139+
140+
- name: Install dependencies
141+
if: ${{ matrix.pkgs }}
142+
run: sudo apt install -y ${{ matrix.pkgs }}
140143

141144
- name: Test
142-
run: ./dev/${{ matrix.script }}
145+
run: |
146+
make env
147+
source .venv/bin/activate
148+
./dev/${{ matrix.script }}
143149
144150
145151
VerifyTestSuites:

0 commit comments

Comments
 (0)