Skip to content

Commit 88a5d46

Browse files
[WIP] Installing Packages for CPP Format Test
1 parent c013edc commit 88a5d46

File tree

1 file changed

+11
-3
lines changed

1 file changed

+11
-3
lines changed

.github/workflows/test.yml

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -128,15 +128,23 @@ jobs:
128128
fail-fast: false
129129
matrix:
130130
include:
131-
- { name: 'C/C++', script: 'check-format.sh' , with_python: 'yes', pkgs: 'clang-format-18' }
132-
- { name: 'Python', script: 'check-format-py.sh', with_python: 'yes', pkgs: '' }
133-
- { name: 'Python Lint', script: 'pylint_check.py' , with_python: 'yes', pkgs: '' }
131+
- { name: 'C/C++', script: 'check-format.sh' , with_python: 'yes', all_vtr_pkgs: 'yes', pkgs: 'clang-format-18' }
132+
- { name: 'Python', script: 'check-format-py.sh', with_python: 'yes', all_vtr_pkgs: 'no', pkgs: '' }
133+
- { name: 'Python Lint', script: 'pylint_check.py' , with_python: 'yes', all_vtr_pkgs: 'no', pkgs: '' }
134134
name: 'F: ${{ matrix.name }}'
135135
steps:
136136

137137
- uses: actions/checkout@v4
138138
# NOTE: We do not need sub-modules. We do not check sub-modules for formatting.
139139

140+
# In order to perform a cpp format of VTR, make format is used which requires
141+
# all of the vpr dependencies to be installed.
142+
# TODO: This should be fixed so this test is not as heavy. We do not need
143+
# these dependencies to perform a simple format.
144+
- name: Install dependencies
145+
if: ${{ matrix.all_vtr_pkgs == 'yes' }}
146+
run: ./install_apt_packages.sh
147+
140148
# TODO: This should be on the same version of Python as would be found on
141149
# Ubuntu 24.04 (3.12.3); however that version has some linting errors.
142150
# - The version of Pylint in requirements.txt is not compatible with

0 commit comments

Comments
 (0)