Merge pull request #44 from SundayWindy/feat/pre-commit

feat(pre-commit): add pre-commit

Author: ruicore

.github/workflows/.precommit.yaml

@@ -0,0 +1,24 @@
+name: pre-commit
+
+on:
+  pull_request:
+  push:
+    branches: [main]
+
+jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+    steps:
+    - name: Check out code
+      uses: actions/checkout@v3
+
+    - name: Set up Python
+      uses: actions/setup-python@v3
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install pre-commit flit && flit install --symlink
+
+    - name: Run pre-commit hooks
+      run: pre-commit run --all-files

excelalchemy/core/alchemy.py

@@ -353,7 +353,7 @@ def _validate_header(self, input_excel_name: str) -> ValidateHeaderResult:
         unrecognized = list(set(input_labels) - set(x.label for x in self.ordered_field_meta))
 
         missing_primary, missing_required = [], []
-        if self.config == ImportMode.UPDATE:
+        if self.config.import_mode == ImportMode.UPDATE:
             missing_primary = list(set(primary_labels) - set(input_labels))
 
         missing_required = list(set(required_labels) - set(input_labels) - set(missing_primary))

pyproject.toml

@@ -32,7 +32,7 @@ development = [
     "mypy",
     "pylint",
     "pre-commit",
-    "pyright",
+    "pyright==1.1.299",
     "pytest",
     "coverage",
     "pytest-cov",