Skip to content

Conversation

@EndlessTrax
Copy link
Owner

@EndlessTrax EndlessTrax commented Jul 12, 2025

  • Update all dependencies
  • Fix type hint errors
  • Fix pytest warning
  • Add python 3.13 to supported versions
  • Bump version

@EndlessTrax EndlessTrax changed the title Update dependencies and minor fixes Add Python 3.13 support and update dependencies Jul 12, 2025
@EndlessTrax EndlessTrax requested a review from Copilot July 12, 2025 14:13

This comment was marked as outdated.

@EndlessTrax EndlessTrax requested a review from Copilot July 12, 2025 14:25
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 adds support for Python 3.13, bumps the package version, and updates type hints and dependency workflows.

  • Bump version to 0.8.1 and add Python 3.13 classifier in pyproject.toml
  • Fix type hints in link_checker.py and tighten exception handling
  • Update CLI return annotation and parser features
  • Extend CI and release workflows to include Python 3.13

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pyproject.toml Bump version and add Python 3.13 to supported versions
pyanchor/link_checker.py Replace re import with Optional, adjust method signature, add None guard, tighten exception clause
pyanchor/cli.py Change print_results return annotation and update parser feature to lxml-xml
.github/workflows/run_tests.yml Include Python 3.13 in test matrix
.github/workflows/release.yml Install Python 3.13 in release workflow
Comments suppressed due to low confidence (1)

pyanchor/cli.py:11

  • The function signature now returns a generic tuple, but the docstring still implies an int return. Please update the docstring to describe the actual return structure and consider using a more specific type annotation (e.g., tuple[int, int]).
def print_results(links: dict, verbose: bool) -> tuple:

@EndlessTrax EndlessTrax merged commit f317a1d into main Jul 12, 2025
12 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