Skip to content

Conversation

@alexmojaki
Copy link
Owner

No description provided.

@alexmojaki alexmojaki requested a review from Copilot September 6, 2025 12:56

This comment was marked as outdated.

@alexmojaki alexmojaki requested a review from Copilot September 6, 2025 13:43
Copy link

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 upgrades the codebase to support Python 3.13 and drops support for older Python versions (2.7, 3.5-3.7). The changes modernize the codebase by removing Python 2 compatibility code, updating test configurations, and addressing Python 3.13-specific behavioral changes.

  • Modernize tox configuration and CI pipeline to test Python 3.8-3.13
  • Remove Python 2 compatibility code and legacy imports
  • Update test expectations for Python 3.10+ encoding error handling changes

Reviewed Changes

Copilot reviewed 27 out of 29 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tox.ini Updates Python version matrix and test runner configuration
tests/test_utils.py Adapts encoding error tests for Python 3.10+ behavioral changes
tests/test_interface.py Updates Selenium usage and removes server management code
tests/test_birdseye.py Removes Python 2-specific tests and modernizes imports
setup.cfg Updates Python version classifiers and dependencies
birdseye/utils.py Removes Python 2 compatibility utilities and encoding handling
birdseye/tracer.py Simplifies AST node creation for Python 3.8+
birdseye/bird.py Removes Python 2 compatibility and updates AST handling
misc/test.sh Updates test script for modern Python tooling

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@alexmojaki alexmojaki merged commit 514114b into master Sep 6, 2025
6 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