Skip to content

Commit 0516b9b

Browse files
committed
Remove redundant install-test job from CI workflow
The install-test job was removed from the GitHub Actions workflow because it duplicated functionality already provided by the main test job. This simplifies the workflow and reduces unnecessary runs across multiple operating systems. Also fixed a quoting issue with the install prefix on Windows.
1 parent e940916 commit 0516b9b

File tree

1 file changed

+1
-13
lines changed

1 file changed

+1
-13
lines changed

templates/.github/workflows/ci.yml.in

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -53,18 +53,6 @@ jobs:
5353
- name: Test
5454
run: ctest --preset=test
5555

56-
install-test:
57-
name: Test Installation (${{ matrix.os }})
58-
runs-on: ${{ matrix.os }}
59-
needs: test
60-
strategy:
61-
fail-fast: false
62-
matrix:
63-
os: [ubuntu-latest, macos-latest, windows-latest]
64-
65-
steps:
66-
- uses: actions/checkout@v5
67-
6856
- name: Build and Install
6957
run: |
7058
cmake --preset=default
@@ -89,7 +77,7 @@ jobs:
8977
EOF
9078

9179
# Convert paths to forward slashes for CMake (works on all platforms)
92-
INSTALL_PREFIX=$(echo "${{ runner.temp }}/install" | sed 's|\\|/|g')
80+
INSTALL_PREFIX=$(echo '${{ runner.temp }}/install' | sed 's|\\|/|g')
9381

9482
# Test find_package with CMAKE_PREFIX_PATH
9583
cmake -B build -S . -DCMAKE_PREFIX_PATH="${INSTALL_PREFIX}"

0 commit comments

Comments
 (0)