chore: update deps, switch to Python 3.14, remove pyright#81
Conversation
|
Unfortunately no stable pre-commit for ty available just yet. |
There was a problem hiding this comment.
Pull request overview
This PR upgrades the project from Python 3.13 to Python 3.14, replaces pyright with ty for type checking, and switches from pre-commit to prek for git hook management. The changes also update dependency versions to use more flexible version constraints.
- Updates Python version from 3.13 to 3.14 across all configuration files
- Replaces pyright with ty for type checking
- Migrates from pre-commit to prek for pre-commit hook management
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| ruff.toml | Updates target Python version to 3.14 |
| pyproject.toml | Updates Python version requirement, replaces pre-commit with prek, removes pyright configuration |
| README.md | Updates documentation to reflect Python 3.14, ty, and prek usage |
| Makefile | Updates lint command to use prek and adds ty check |
| Dockerfile | Updates base image to Python 3.14 |
| .pre-commit-config.yaml | Migrates to builtin hooks, updates hook versions, removes pyright hook |
| .github/workflows/python-app.yml | Updates CI to use Python 3.14 and prek |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| rev: v0.14.9 | ||
| hooks: | ||
| - id: ruff | ||
| - id: ruff-check |
There was a problem hiding this comment.
The hook ID has changed from 'ruff' to 'ruff-check'. Verify that this is the correct hook ID for the version of ruff-pre-commit being used (v0.14.9), as this may break pre-commit functionality if the hook ID doesn't exist in that version.
| - id: ruff-check | |
| - id: ruff |
|
Niiice, I have been waiting for this. What about this? |
I did notice it, but I am willing to wait for an official (more stable) release, see astral-sh/ty#269 |
|
Ok, sounds good! Please merge when ready. |
Switched to ty and close #72
Also switched to prek from pre-commit.