Updated CI to use pyproject.toml, ruff linter

.github/workflows/python-package.yml

@@ -2,34 +2,35 @@ name: tests
 
 on:
   push:
-    branches: [ master ]
+    branches:
+      - '*'
   pull_request:
-    branches: [ master ]
+    branches:
+      - '*'
 
 jobs:
   build:
 
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.8", "3.9", "3.10", "3.11"]
+        python-version: ["3.8"]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v4
       with:
         python-version: ${{ matrix.python-version }}
     - name: Install dependencies
       run: |
-        python -m pip install --upgrade pip wheel setuptools pytest types-setuptools mypy
-    - name: Run mypy
+        python -m pip install ".[dev]"
+    - name: Run ruff linter
       run: |
-        mypy iceaddr/*.py
+        ruff src/iceaddr
     - name: Test with pytest
       run: |
-        python -m pytest
-
+        python -m pytest -vvv
 
 
 

pyproject.toml

@@ -33,15 +33,16 @@ Repository = "https://github.com/sveinbjornt/iceaddr"
 [project.optional-dependencies]
 # Dev dependencies
 dev = [
-    "pytest>=7.2.1",
     # Building database
-    "fiona>=1.9.4",    # for GIS data
+    "fiona>=1.9.4",    # for processing GIS data
     "humanize>=0.5.1", # for progress bar
     "reynir>=3.1.0",   # for parsing and declining placenames
-    # Pre-commit hook
+    # Linting, testing
+    "pytest>=7.2.1",
     "pre-commit>=3.3.3",
     "black>=23.7.0",
     "mypy>=1.5.1",
+    "types-setuptools>=68.2.0",
     "ruff>=0.0.285",
     "coverage[toml]>=7.3.1",
 ]
@@ -135,7 +136,7 @@ section-order = [
 
 [tool.ruff.isort.sections]
 # Have typing libraries above other imports
-typehints = ["typing", "typing_extensions", "types"]
+typehints = ["typing", "typing_extensions", "types", "types-setuptools"]
 
 # *** Build system configuration ***
 

src/iceaddr/postcodes.py

@@ -1428,9 +1428,7 @@ def _filter_postcodes(key: str, searchstr: str, partial: bool = False) -> List[i
     for k, v in POSTCODES.items():
         nf = v[k1].lower()
         tgf = v[k2].lower()
-        if partial and (nf.startswith(p) or tgf.startswith(p)):
-            matches.append(k)
-        elif nf == p or tgf == p:
+        if (partial and (nf.startswith(p) or tgf.startswith(p))) or p in (nf, tgf):
             matches.append(k)
 
     return matches