Skip to content

Update tox.ini to remove Python 3.8 and change mypy version - #134

Merged
auvipy merged 1 commit into
masterfrom
ci-1
Feb 18, 2026
Merged

Update tox.ini to remove Python 3.8 and change mypy version#134
auvipy merged 1 commit into
masterfrom
ci-1

Conversation

@auvipy

@auvipy auvipy commented Feb 18, 2026

Copy link
Copy Markdown
Collaborator

Removed Python 3.8 from envlist and updated mypy basepython to 3.12.

Removed Python 3.8 from envlist and updated mypy basepython to 3.12.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes Python 3.8 from the supported versions in the tox configuration and updates the mypy test environment to use Python 3.12. The removal of Python 3.8 aligns with the codebase's use of PEP 585-style type hints (e.g., dict[str, Any]) which require Python 3.9+. Additionally, the PR fixes a pre-existing typo where mypy was configured to use python3.3 instead of a modern Python version.

Changes:

  • Removed Python 3.8 from the tox envlist
  • Removed the gh-actions mapping for Python 3.8
  • Updated mypy basepython from 3.3 to 3.12 (fixing a typo)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tox.ini
[gh-actions]
python =
3.8: py38
python =

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is trailing whitespace at the end of this line. This should be removed for consistency and cleaner formatting.

Suggested change
python =
python =

Copilot uses AI. Check for mistakes.
@auvipy
auvipy merged commit 9a05fe9 into master Feb 18, 2026
46 checks passed
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