build: enable format, warning, and coverage by default (#56)

* build: remove `CHECK_FORMAT` option

* build: remove `CHECK_WARNING` option

* build: remove `CHECK_COVERAGE` option

* build: modify `fix-format` target to be called manually

Author: threeal

.github/workflows/build.yml

@@ -16,10 +16,12 @@ jobs:
         run: |
           cmake . \
             -B build \
-            -D BUILD_TESTING=ON \
-            -D CHECK_FORMAT=ON \
-            -D CHECK_WARNING=ON \
-            -D CHECK_COVERAGE=ON
+            -D BUILD_TESTING=ON
+
+      - name: Check formatting
+        run: |
+          cmake --build build --target fix-format
+          git diff --exit-code HEAD
 
       - name: Build project
         run: cmake --build build
@@ -49,8 +51,7 @@ jobs:
           cmake . `
             -B build `
             -D CMAKE_CXX_COMPILER=cl `
-            -D BUILD_TESTING=ON `
-            -D CHECK_WARNING=ON
+            -D BUILD_TESTING=ON
 
       - name: Build project
         run: cmake --build build

CMakeLists.txt

@@ -14,16 +14,10 @@ target_compile_features(errors PRIVATE cxx_std_20)
 
 # Check if this project is the main project
 if(CMAKE_CURRENT_SOURCE_DIR STREQUAL CMAKE_SOURCE_DIR)
-  option(CHECK_FORMAT "Enable source code formatting check" OFF)
-  option(CHECK_WARNING "Enable static analysis warning check" OFF)
-  option(CHECK_COVERAGE "Enable test coverage check" OFF)
   option(BUILD_DOCS "Enable documentations build" OFF)
 
   # Import Format.cmake to format source code
-  if(CHECK_FORMAT)
-    cpmaddpackage("gh:threeal/Format.cmake#auto-install-cmake-format")
-    add_dependencies(errors fix-format)
-  endif()
+  cpmaddpackage("gh:threeal/Format.cmake#auto-install-cmake-format")
 
   if(BUILD_TESTING)
     enable_testing()
@@ -43,16 +37,14 @@ if(CMAKE_CURRENT_SOURCE_DIR STREQUAL CMAKE_SOURCE_DIR)
 
   foreach(TARGET IN LISTS TARGETS)
     # Statically analyze code by checking for warnings
-    if(CHECK_WARNING)
-      if(MSVC)
-        target_compile_options(${TARGET} PRIVATE /WX /permissive- /W4 /w14640 /EHsc)
-      else()
-        target_compile_options(${TARGET} PRIVATE -Werror -Wall -Wextra -Wnon-virtual-dtor -Wpedantic)
-      endif()
+    if(MSVC)
+      target_compile_options(${TARGET} PRIVATE /WX /permissive- /W4 /w14640 /EHsc)
+    else()
+      target_compile_options(${TARGET} PRIVATE -Werror -Wall -Wextra -Wnon-virtual-dtor -Wpedantic)
     endif()
 
     # Enable support to check for test coverage
-    if(BUILD_TESTING AND CHECK_COVERAGE AND NOT MSVC)
+    if(BUILD_TESTING AND NOT MSVC)
       target_compile_options(${TARGET} PRIVATE --coverage -O0)
       target_link_options(${TARGET} PRIVATE --coverage)
     endif()