Skip to content

Commit

Permalink
Simplify GitHub Actions workflow test job configuration
Browse files Browse the repository at this point in the history
- Updated `test.yml` to simplify test job steps.
- Removed build type matrix: `Release`, `Debug`.
- Expanded OS matrix: added `ubuntu-24.04`, `macos-14`, `macos-15`.
- Replaced multi-platform CMake configurations with `make` command.

Streamlining the workflow reduces complexity and maintenance overhead. Using the
expanded OS options ensures broader testing coverage. Switching to `make` aligns
with our build strategy and removes redundant configuration steps.
  • Loading branch information
horta committed Nov 7, 2024
1 parent c3dccef commit 5359d37
Showing 1 changed file with 6 additions and 37 deletions.
43 changes: 6 additions & 37 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,46 +4,15 @@ on: push

jobs:
test:
name: Test on ${{ matrix.os }} with build type ${{ matrix.build_type }}
name: Test on ${{ matrix.os }}
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ubuntu-22.04, macos-12, macos-13, windows-2022]
build_type: [Release, Debug]
os: [ubuntu-22.04, ubuntu-24.04, macos-13, macos-14, macos-15, windows-2022]

steps:
- uses: actions/checkout@v4

- name: Configure CMake
if: runner.os == 'Linux'
run: >
cmake
-B ${{github.workspace}}/build
-DCMAKE_BUILD_TYPE=${{matrix.build_type}}
-DUSE_SANITIZER='Address;Undefined'
-DENABLE_ALL_WARNINGS=ON
- name: Configure CMake
if: runner.os == 'macOS'
run: >
cmake
-B ${{github.workspace}}/build
-DCMAKE_BUILD_TYPE=${{matrix.build_type}}
-DENABLE_ALL_WARNINGS=ON
- name: Configure CMake
if: runner.os == 'Windows'
run: >
cmake
-B ${{github.workspace}}/build
-DCMAKE_BUILD_TYPE=${{matrix.build_type}}
- name: Build
run: >
cmake
--build ${{github.workspace}}/build
--config ${{matrix.build_type}}
- name: Test
working-directory: ${{github.workspace}}/build
run: ctest -C ${{matrix.build_type}} --output-on-failure
- name: make
run: make
- name: make check
run: make check

0 comments on commit 5359d37

Please sign in to comment.