Skip to content

fix: Python 3.8 compatibility#63

Merged
alaurenzi merged 2 commits into
masterfrom
copilot/ensure-compatibility-with-python-38
Jun 3, 2026
Merged

fix: Python 3.8 compatibility#63
alaurenzi merged 2 commits into
masterfrom
copilot/ensure-compatibility-with-python-38

Conversation

Copilot AI commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Two type annotation constructs introduced after Python 3.8 were used at module/function scope, causing TypeError at import time on Python 3.8.

Changes

  • sys_deps.py: Add from __future__ import annotations (PEP 563) — defers annotation evaluation so str | None (PEP 604, 3.10+) and list[str] (PEP 585, 3.9+) are never evaluated as expressions at runtime
  • parser.py: Use raw strings for regex patterns to eliminate invalid escape sequences (\[, \s) that produce SyntaxWarning
  • .github/workflows/python-tests.yml: Add Python 3.8 to the CI test matrix to explicitly verify compatibility going forward

@alaurenzi

Copy link
Copy Markdown
Contributor

@copilot Add python 3.8 to the test matrix (gh workflow)

Copilot AI commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Add python 3.8 to the test matrix (gh workflow)

Done — added '3.8' to the python-version matrix in .github/workflows/python-tests.yml (commit ci: add Python 3.8 to test matrix).

Copilot AI requested a review from alaurenzi June 3, 2026 10:14
@alaurenzi alaurenzi merged commit 84be4a9 into master Jun 3, 2026
4 checks passed
@alaurenzi alaurenzi deleted the copilot/ensure-compatibility-with-python-38 branch June 3, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants