Conversation
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
CodSpeed Performance ReportMerging this PR will not alter performanceComparing Summary
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #140 +/- ##
=======================================
Coverage 88.29% 88.29%
=======================================
Files 3 3
Lines 94 94
=======================================
Hits 83 83
Misses 11 11 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Pull request overview
This PR adds "ty" as a static analysis tool to the project's development workflow, integrating it alongside existing linting tools like mypy, ruff, and prek.
Changes:
- Added
ty>=0.0.12to development dependencies in pyproject.toml - Integrated
ty check .command into the nox linting session - Added a dedicated "Run ty" step in the GitHub Actions CI workflow
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| pyproject.toml | Added ty package to dev dependency group |
| uv.lock | Added ty package metadata and wheel information for multiple platforms |
| noxfile.py | Added ty check command to the lints session |
| .github/workflows/python-ci.yaml | Added ty check step to the lint job |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| session.run("ruff", "check", "--fix", ".") | ||
| session.run("ruff", "format", ".") | ||
| session.run("mypy", "--strict", "src/") | ||
| session.run("ty", "check", ".") |
There was a problem hiding this comment.
The noxfile adds a new static analysis tool 'ty', but there is no configuration for this tool in pyproject.toml (unlike other tools such as [tool.mypy] or [tool.ruff]). If 'ty' requires configuration, consider adding a [tool.ty] section to pyproject.toml with appropriate settings. If no configuration is needed, this comment can be disregarded.
| session.run("ty", "check", ".") |
No description provided.