Skip to content

Commit

Permalink
ci: convert to use CMake
Browse files Browse the repository at this point in the history
Convert the CI to use CMake instead of Makefiles. This will allow us to
wire up support for Windows in a subsequent commit.
  • Loading branch information
pks-t committed Sep 5, 2024
1 parent f7bd3e9 commit 2e5c308
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,7 @@ jobs:
uses: actions/checkout@v2
- name: Build
run: |
cd test
make
mkdir build
cd build
cmake ..
cmake --build .
5 changes: 3 additions & 2 deletions test/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,9 @@ set_target_properties(clar_test PROPERTIES
C_EXTENSIONS OFF
)

# MSVC generates all kinds of warnings.
if(NOT "${CMAKEC_COMPILER_ID}" MATCHES "MSVC")
# MSVC generates all kinds of warnings. We may want to fix these in the future
# and then unconditionally treat warnings as errors.
if (NOT MSVC)
set_target_properties(clar_test PROPERTIES
COMPILE_WARNING_AS_ERROR ON
)
Expand Down

0 comments on commit 2e5c308

Please sign in to comment.