Enable warnings as errors

Author: ashtum

.github/workflows/ci.yml

@@ -53,6 +53,10 @@ jobs:
             cxxstd: 17,20
             os: ubuntu-24.04
             install: g++-13
+          - compiler: g++-14
+            cxxstd: 17,20,23
+            os: ubuntu-24.04
+            install: g++-14
           - compiler: clang++-6.0
             cxxstd: 17
             os: ubuntu-latest
@@ -115,16 +119,24 @@ jobs:
             cxxstd: 17,20,23
             os: ubuntu-24.04
             install: clang-18
+          - compiler: clang++-19
+            cxxstd: 17,20,23
+            os: ubuntu-24.04
+            install: clang-19
+          - compiler: clang++-20
+            cxxstd: 17,20,23
+            os: ubuntu-24.04
+            install: clang-20
 # macos
-          - compiler: clang++
-            os: macos-13
-            cxxstd: 17,20
           - compiler: clang++
             os: macos-14
             cxxstd: 17,20
           - compiler: clang++
             os: macos-15
             cxxstd: 17,20,23
+          - compiler: clang++
+            os: macos-26
+            cxxstd: 17,20,23
 
     runs-on: ${{matrix.os}}
     container:

CMakeLists.txt

@@ -14,12 +14,6 @@ target_include_directories(lazycsv INTERFACE include/)
 
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
 
-if(MSVC)
-  add_compile_options(/W4)
-elseif(CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
-  add_compile_options(-Wall -Wfatal-errors -Wextra -Wnon-virtual-dtor -pedantic)
-endif()
-
 # Install headers
 install(DIRECTORY "${CMAKE_SOURCE_DIR}/include/"
   DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}

test/CMakeLists.txt

@@ -2,6 +2,12 @@ add_executable(main main.cpp)
 
 target_link_libraries(main lazycsv)
 
+if(MSVC)
+  add_compile_options(/W4 /WX)
+elseif(CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
+  add_compile_options(-Wall -Wfatal-errors -Wextra -Wnon-virtual-dtor -pedantic -Werror)
+endif()
+
 target_include_directories(main SYSTEM PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/doctest)
 
 add_custom_command(