Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #324 +/- ##
=======================================
Coverage 98.41% 98.41%
=======================================
Files 11 11
Lines 1010 1010
Branches 120 120
=======================================
Hits 994 994
Misses 9 9
Partials 7 7
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Updates linting/formatting configuration to standardize tooling around Ruff and allow a 120-character line length, along with reformatting affected code to match the new style.
Changes:
- Move line-length configuration to Ruff and increase it to 120; remove Black configuration.
- Update pre-commit hooks to use
mdformat-ruffand rename the Ruff lint hook. - Apply formatting-only changes across src/, tests/, and examples/ to align with the updated formatting rules.
Reviewed changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.pre-commit-config.yaml |
Switch mdformat plugin to mdformat-ruff and rename Ruff hook for clarity. |
pyproject.toml |
Remove Black config and set Ruff line-length = 120. |
src/cachier/core.py |
Formatting-only changes to function signatures and wrapped expressions. |
src/cachier/config.py |
Formatting-only changes to helper function definitions and warning strings. |
src/cachier/cores/base.py |
Formatting-only change to default-parameter update call. |
src/cachier/cores/memory.py |
Formatting-only changes to method signatures and comprehensions. |
src/cachier/cores/mongo.py |
Formatting-only changes to raised error formatting and query formatting. |
src/cachier/cores/pickle.py |
Formatting-only changes to wrapped expressions and method signatures. |
src/cachier/cores/redis.py |
Formatting-only changes to type annotations and warning strings. |
src/cachier/cores/sql.py |
Formatting-only changes to table args, error strings, and wrapped expressions. |
examples/redis_example.py |
Formatting-only changes to Redis client construction call. |
tests/test_base_core.py |
Formatting-only changes to core construction calls. |
tests/test_cleanup.py |
Formatting-only changes to string replacement chain. |
tests/test_defaults.py |
Formatting-only changes to pytest.deprecated_call blocks. |
tests/test_general.py |
Formatting-only change to pytest.param formatting. |
tests/test_main.py |
Formatting-only changes to Click runner invocation formatting. |
tests/test_memory_core.py |
Formatting-only changes to threading calls and hash helper formatting. |
tests/test_mongo_core.py |
Formatting-only changes to strings, thread creation, and monkeypatch calls. |
tests/test_pickle_core.py |
Formatting-only changes to helper calls, constants, and prints. |
tests/test_redis_core.py |
Formatting-only changes to Redis client construction, debug prints, and decorators. |
tests/test_redis_core_exceptions.py |
Formatting-only changes to fixtures, warnings contexts, and dict literals. |
tests/test_sql_core.py |
Formatting-only change to pytest.raises usage formatting. |
This pull request updates linting and formatting configurations to improve code consistency and streamline tooling. The main changes involve switching formatting plugins, updating pre-commit hooks, and adjusting line length settings.
Pre-commit configuration updates:
mdformat-blackplugin withmdformat-ruffin themdformatpre-commit hook to standardize markdown formatting using Ruff conventions.ruffpre-commit hook toruff-checkfor clarity and consistency.Formatting and linting settings:
[tool.black]section and itsline-lengthsetting frompyproject.toml, consolidating line length configuration under Ruff.line-lengthsetting for Ruff from 79 to 120 characters to allow for longer lines and improve readability.