🔨Upgrade pre-commit hooks and replace some with Ruff #8716
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #8716 +/- ##
==========================================
+ Coverage 87.47% 89.32% +1.84%
==========================================
Files 2018 1712 -306
Lines 79384 68498 -10886
Branches 1389 646 -743
==========================================
- Hits 69444 61183 -8261
+ Misses 9545 7146 -2399
+ Partials 395 169 -226
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Contributor
🧪 CI InsightsHere's what we observed from your CI run for 998db0d. ✅ Passed Jobs With Interesting Signals
|
sanderegg
force-pushed
the
maintenance/tooling2
branch
from
d541fb2 to
49d36a5
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR modernizes the Python tooling setup by migrating from Black to Ruff as the primary code formatter and linter. Ruff is a faster, all-in-one tool that combines the functionality of multiple tools (Black, isort, pyupgrade, pycln, flake8).
- Replaces Black with Ruff for code formatting across the repository
- Consolidates pre-commit hooks by removing redundant tools (pyupgrade, pycln, isort, Black)
- Updates configuration files and documentation to reflect the tooling change
Reviewed changes
Copilot reviewed 16 out of 18 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.ruff.toml |
Expanded configuration with formatting rules, exclusions, and linting options that replace Black/isort settings |
.pre-commit-config.yaml |
Added Ruff hooks and removed deprecated tooling (Black, isort, pyupgrade, pycln) |
.isort.cfg |
Removed as isort functionality is now handled by Ruff |
.flake8 |
Removed as flake8 functionality is now handled by Ruff |
README.md |
Updated badge from Black to Ruff |
CONTRIBUTING.md |
Updated documentation to reference Ruff instead of Black |
docs/coding-conventions.md |
Updated coding guidelines to mention Ruff instead of Black |
.github/instructions/python.instructions.md |
Updated instructions to use Ruff for formatting |
.vscode/settings.template.json |
Configured Ruff as default Python formatter with format-on-save |
.vscode/launch.template.json |
Removed redundant comments |
.vscode/extensions.json |
Removed Black formatter extension, kept trailing comma |
packages/models-library/Makefile |
Changed from black to ruff format command |
packages/service-integration/Makefile |
Changed from black to ruff format command |
scripts/openapi/openapi_python_server_codegen.sh |
Fixed shebang typo |
services/web/server/docker/healthcheck.py |
Minor code improvements: added HTTP_OK constant and noqa comment |
services/web/server/tests/data/server_docker_prod_app_config-unit.json |
Removed trailing comma from JSON |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
sanderegg
changed the title
sanderegg
requested review from
GitHK,
giancarloromeo,
matusdrobuliak66 and
pcrespov
as code owners
GitHK
approved these changes
Jan 6, 2026
Contributor
GitHK
left a comment
GitHK
left a comment
There was a problem hiding this comment.
all right let's see how this behaves
sanderegg
added
t:maintenance
Member
Author
|
@mergify queue |
Contributor
🟠 Waiting for conditions to matchDetails
|
Contributor
Merge Queue Status🟠 Waiting for queue conditions Required conditions to enter a queue
|
sanderegg
force-pushed
the
maintenance/tooling2
branch
from
9070d64 to
998db0d
Compare
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What do these changes do?
replace pre-commit hooks such as black, pyupgrade, isort and pycln by ruff.
speed increase while pre-commiting, and removing weird jumps between black and ruff.
also added the format on save behavior since it is very fast.
Related issue/s
How to test
Dev-ops