Skip to content

Commit

Permalink
Merge pull request #5 from s-bose7/build-ci
Browse files Browse the repository at this point in the history
Fix: GitHub workflow matrix usage
(Gtest dependency issue not resolved)
  • Loading branch information
s-bose7 authored Jun 19, 2024
2 parents 6c63ea0 + 83e7389 commit 7f2371d
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,11 @@ on:

jobs:
build:
runs-on: ${{ build_ps.os }}
runs-on: ${{ matrix.os }}

strategy:
fail-fast: false

build_ps:
matrix:
os: [ubuntu-latest]
build_type: [Release]
c_compiler: [gcc]
Expand Down Expand Up @@ -50,13 +49,13 @@ jobs:
- name: Configure CMake
run: |
cmake -B ${{ steps.strings.outputs.build-output-dir }} \
-DCMAKE_CXX_COMPILER=${{ build_ps.cpp_compiler }} \
-DCMAKE_C_COMPILER=${{ build_ps.c_compiler }} \
-DCMAKE_BUILD_TYPE=${{ build_ps.build_type }} \
-DCMAKE_CXX_COMPILER=${{ matrix.cpp_compiler }} \
-DCMAKE_C_COMPILER=${{ matrix.c_compiler }} \
-DCMAKE_BUILD_TYPE=${{ matrix.build_type }} \
-S ${{ github.workspace }}
- name: Build
run: cmake --build ${{ steps.strings.outputs.build-output-dir }} --config ${{ build_ps.build_type }}
run: cmake --build ${{ steps.strings.outputs.build-output-dir }} --config ${{ matrix.build_type }}

- name: Run Tests
run: cd ${{ steps.strings.outputs.build-output-dir }} && ctest

0 comments on commit 7f2371d

Please sign in to comment.