Skip to content

Commit

Permalink
Merge pull request #41 from dmatveev/dm/fix-win-ci
Browse files Browse the repository at this point in the history
CI: Remove Ninja as it is not listed in Windows image
  • Loading branch information
dmatveev authored Oct 10, 2023
2 parents 639d3cd + d295d8f commit 0cad24f
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions .github/workflows/workflow.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,9 @@ jobs:
mkdir build && cd build
if [ "$RUNNER_OS" == "Windows" ]; then
cmake -DENABLE_ADE_TESTING=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_STANDARD=${{ matrix.version }} -G Ninja ..
cmake --build . --target GTest
cmake --build .
cmake -DENABLE_ADE_TESTING=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_STANDARD=${{ matrix.version }} ..
cmake --build . --target GTest --config Release
cmake --build . --config Release
elif [ "$RUNNER_OS" == "Linux" ]; then
cmake -DENABLE_ADE_TESTING=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_STANDARD=${{ matrix.version }} ..
Expand All @@ -40,10 +40,6 @@ jobs:
exit 1
fi
- name: ADE-Test Stage
shell: bash
run: ./build/bin/ade-tests

- name: CMN-Test Stage
- name: Test Stage
shell: bash
run: ./build/bin/common-tests
run: ctest -C Release

0 comments on commit 0cad24f

Please sign in to comment.