Skip to content

Commit

Permalink
chore: re-enabled test matrix
Browse files Browse the repository at this point in the history
  • Loading branch information
Goldziher committed Feb 19, 2025
1 parent 43bf7b2 commit 40f5517
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 11 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,8 @@ jobs:
test:
strategy:
matrix:
os: [ ubuntu-latest ]
python: [ " 3.9" ]
#os: [ ubuntu-latest, macOS-latest, windows-latest ]
#python: [" 3.9", "3.10", "3.11", "3.12", "3.13" ]
os: [ ubuntu-latest, macOS-latest, windows-latest ]
python: [" 3.9", "3.10", "3.11", "3.12", "3.13" ]
runs-on: ${{ matrix.os }}
timeout-minutes: 20
steps:
Expand Down
2 changes: 1 addition & 1 deletion kreuzberg/_string.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ def safe_decode(byte_data: bytes, encoding: str | None = None) -> str:
encodings = [encoding, detect(byte_data).get("encoding", ""), "utf-8"]

for enc in [e for e in encodings if e]: # pragma: no cover
with suppress(UnicodeDecodeError):
with suppress(UnicodeDecodeError, LookupError):
return byte_data.decode(enc)

# If all encodings fail, fall back to latin-1 which can handle any byte
Expand Down
6 changes: 0 additions & 6 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,6 @@
import pytest


@pytest.fixture
def anyio_backend() -> str:
"""override anyio to test only against asyncio"""
return "asyncio"


@pytest.fixture(scope="session")
def searchable_pdf() -> Path:
return Path(__file__).parent / "source" / "searchable.pdf"
Expand Down

0 comments on commit 40f5517

Please sign in to comment.