Skip to content

Commit 7d38cb9

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

File tree

1 file changed

+23
-7
lines changed

1 file changed

+23
-7
lines changed

.github/workflows/test.yml

Lines changed: 23 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -122,24 +122,40 @@ 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' , with_python: 'no', pkgs: 'clang-format-18' }
126+
- { name: 'Python', script: 'check-format-py.sh', with_python: 'yes', pkgs: '' }
127+
- { name: 'Python Lint', script: 'pylint_check.py' , with_python: 'yes', pkgs: '' }
128128
name: 'F: ${{ matrix.name }}'
129129
steps:
130130

131+
- uses: actions/checkout@v4
132+
# NOTE: We do not need sub-modules. We do not check sub-modules for formatting.
133+
131134
# TODO: This should be on the same version of Python as would be found on
132135
# Ubuntu 24.04 (3.12.3); however that version has some linting errors.
133136
- uses: actions/setup-python@v5
137+
if: ${{ matrix.with_python == 'yes' }}
134138
with:
135139
python-version: 3.10.10
136140

137-
- uses: actions/checkout@v4
138-
with:
139-
submodules: 'true'
141+
- name: Build Python Virtual Env
142+
run: |
143+
python -m venv .venv
144+
145+
- name: Install Python Requirements
146+
if: ${{ matrix.with_python == 'yes' }}
147+
run: |
148+
source .venv/bin/activate
149+
pip install -r requirements.txt
150+
151+
- name: Install dependencies
152+
if: ${{ matrix.pkgs }}
153+
run: sudo apt install -y ${{ matrix.pkgs }}
140154

141155
- name: Test
142-
run: ./dev/${{ matrix.script }}
156+
run: |
157+
source .venv/bin/activate
158+
./dev/${{ matrix.script }}
143159
144160
145161
VerifyTestSuites:

0 commit comments

Comments
 (0)