Skip to content

Remove from __future__ import annotations#1441

Open
edenhaus wants to merge 1 commit intodevfrom
remove-future-annotations
Open

Remove from __future__ import annotations#1441
edenhaus wants to merge 1 commit intodevfrom
remove-future-annotations

Conversation

@edenhaus
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 19, 2026 22:59
@edenhaus edenhaus added the pr: refactor PR with code refactoring label Feb 19, 2026
Copy link
Contributor

Copilot AI left a comment

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 the from __future__ import annotations import from all Python files in the codebase, as it is no longer necessary for Python 3.14+. The project requires Python >=3.14.0, where postponed evaluation of annotations is the default behavior (PEP 649).

Changes:

  • Removed from __future__ import annotations from 188 Python files across the codebase
  • Removed the ruff isort required-imports configuration that enforced this import
  • Added pyupgrade to pre-commit hooks with --py314-plus to prevent the import from being reintroduced

Reviewed changes

Copilot reviewed 253 out of 253 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Removed required-imports configuration from ruff isort settings
.pre-commit-config.yaml Added pyupgrade hook with --py314-plus flag
deebot_client/**/*.py Removed future annotations import from all source files
tests/**/*.py Removed future annotations import from all test files
scripts/**/*.py Removed future annotations import from all script files

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: refactor PR with code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants