Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #118 +/- ##
=======================================
Coverage 99.79% 99.79%
=======================================
Files 9 9
Lines 1443 1443
=======================================
Hits 1440 1440
Misses 3 3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR updates the Python development tooling by removing black and switching formatting checks to ruff format, alongside bumping the ruff dependency.
Changes:
- Remove
blackfrom the dev dependency set (and lockfile) - Bump
ruffto>=0.15.7,<0.16(lockfile now at0.15.7) - Update CI to run
ruff format --checkinstead ofblack
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
uv.lock |
Drops black and its transitive deps; updates locked ruff to 0.15.7 and adjusts dev metadata accordingly. |
pyproject.toml |
Removes black from the dev dependency group; bumps the ruff version constraint. |
.github/workflows/ci.yml |
Replaces the formatting step command from black to ruff format --check. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [dependency-groups] | ||
| dev = [ | ||
| "coverage>=5.3,<6", | ||
| "ruff>=0.14.0,<0.15", | ||
| "ruff>=0.15.7,<0.16", | ||
| "isort>=5.10.1,<6", | ||
| "black>=24.3.0,<27", | ||
| "nose2>=0.12.0,<0.13", | ||
| ] |
There was a problem hiding this comment.
You removed black from the dev dependency group, but pyproject.toml still contains a [tool.black] configuration section. That config is now effectively dead/unused and can confuse contributors; either remove the [tool.black] section or re-add black as a dev dependency if it’s still intended to be supported.
| @@ -23,7 +23,7 @@ jobs: | |||
| run: uv sync --locked --all-groups | |||
|
|
|||
| - name: Run black | |||
There was a problem hiding this comment.
This step is still labeled “Run black” but it now runs ruff format --check. Renaming the step to match the actual command will make CI output clearer when failures occur.
| - name: Run black | |
| - name: Run ruff format |
No description provided.