Skip to content

Conversation

@spietras
Copy link
Member

@spietras spietras commented Jan 5, 2026

No description provided.

@spietras spietras self-assigned this Jan 5, 2026
@spietras spietras added the cleanup Stay fresh label Jan 5, 2026
@spietras spietras requested a review from Copilot January 5, 2026 12:30
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 updates project dependencies and improves type hints, as suggested by the title "Cleaned up code". The changes include dependency version bumps for production and development packages, more flexible type hints using abstract types, and configuration to suppress testcontainers deprecation warnings.

Key Changes:

  • Updated multiple production dependencies (litestar, typer, uvicorn) and development dependencies (pytest, faker, polyfactory, etc.) to their latest versions
  • Improved type hints by using Sequence and Mapping instead of list and dict for better flexibility
  • Added environment variable to suppress testcontainers deprecation warnings in Nix development shells

Reviewed changes

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

Show a summary per file
File Description
src/uv.lock.jinja Updated lock file with new versions for production dependencies (litestar 2.19.0, typer 0.21.0, uvicorn 0.40.0) and development dependencies (anyio, certifi, faker, pytest, polyfactory, rich-click, tzdata, urllib3), plus transitive dependency updates
src/pyproject.toml.jinja Updated production dependency version constraints to match the lock file updates
src/tests/utils/waiting/conditions.py Improved type hints by changing from list[str] to Sequence[str] and dict[str, str] to Mapping[str, str] for more flexible parameter types
src/tests/utils/containers.py Changed type hint from list[str] to Sequence[str] for command parameter consistency
src/flake.nix.jinja Added PYTHONWARNINGS environment variable to suppress testcontainers deprecation warnings in dev and test shells with reference to upstream issue

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

@spietras spietras merged commit 108a7b8 into main Jan 5, 2026
2 checks passed
@spietras spietras deleted the cleanup branch January 5, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Stay fresh

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants