Cleanups (#259)

.pre-commit-config.yaml

@@ -22,4 +22,3 @@ repos:
     hooks:
       - id: clang-format
         args: [--style=LLVM]
-        files: 'pantab/src/(.*)'

CMakeLists.txt

@@ -7,7 +7,7 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
 if (MSVC)
 else()
-    add_compile_options(-Wall -Wextra)
+    add_compile_options(-Wall -Wextra -Werror)
 endif()
 
 find_package(Python COMPONENTS Interpreter Development.Module NumPy REQUIRED)

asv.conf.json

@@ -24,8 +24,8 @@
   // "install_command": ["in-dir={env_dir} python -mpip install {wheel_file}"],
   // "uninstall_command": ["return-code=any python -mpip uninstall -y {project}"],
   "build_command": [
-      "python -m pip install build",
-      "python -m build --wheel -o {build_cache_dir} {build_dir}",
+    "python -m pip install build",
+    "python -m build --wheel -o {build_cache_dir} {build_dir}",
   ],
 
   // List of branches to benchmark. If not provided, defaults to "master"

src/pantab/CMakeLists.txt

@@ -1,4 +1,10 @@
-nanobind_add_module(libpantab NOMINSIZE libpantab.cpp numpy_datetime.cpp)
+nanobind_add_module(
+  libpantab
+  NOMINSIZE
+  libpantab.cpp
+  reader.cpp
+  writer.cpp
+  numpy_datetime.cpp)
 target_include_directories(libpantab PUBLIC ${Python_NumPy_INCLUDE_DIRS})
 target_link_libraries(libpantab
   PRIVATE Tableau::tableauhyperapi-cxx