Skip to content

Commit 9918c25

Browse files
committed
Disable testing by default
1 parent f820d2c commit 9918c25

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

.github/workflows/main.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,10 @@ jobs:
4040
wget https://apt.llvm.org/llvm.sh
4141
chmod +x llvm.sh
4242
sudo ./llvm.sh ${{ matrix.clang_version }}
43-
- name: Install ninja
44-
run: sudo apt-get install ninja-build
43+
- name: Install ninja and catch2
44+
run: sudo apt-get install ninja-build catch2
4545
- name: configure
46-
run: cmake --preset linux-clang-release
46+
run: cmake --preset linux-clang-release -D REFLECTION_TESTING=ON
4747
- name: build
4848
run: cmake --build --preset linux-clang-release
4949
- name: test
@@ -59,9 +59,9 @@ jobs:
5959
steps:
6060
- uses: actions/checkout@v4
6161
- name: Install dependencies
62-
run: sudo apt-get install ninja-build g++-${{ matrix.gcc_version }}
62+
run: sudo apt-get install ninja-build catch2 g++-${{ matrix.gcc_version }}
6363
- name: configure
64-
run: cmake --preset linux-gcc-release
64+
run: cmake --preset linux-gcc-release -D REFLECTION_TESTING=ON
6565
- name: build
6666
run: cmake --build --preset linux-gcc-release
6767
- name: test

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ install(EXPORT reflection-cpp-targets
5252
# ---------------------------------------------------------------------------
5353
# unit tests
5454

55-
option(REFLECTION_TESTING "Enables building of unittests for reflection-cpp [default: ON]" ON)
55+
option(REFLECTION_TESTING "Enables building of unittests for reflection-cpp [default: OFF]" OFF)
5656
if(REFLECTION_TESTING)
5757
find_package(Catch2 3.4.0 QUIET)
5858
if(NOT Catch2_FOUND)

0 commit comments

Comments
 (0)