Replace flake8+isort+black with `ruff`
Pull Request check-list
Please make sure to review and check all of these items:
- [x] Do tests and lints pass with this change?
- [x] Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
- [ ] Is the new or changed code fully tested?
- [ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
- [ ] Is there an example added to the examples folder (if applicable)?
- [ ] Was the change added to CHANGES file?
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
Description of change
This PR replaces the use of flake8, isort, and black with Ruff.
More ruff rules could (should) be enabled later and the noqas taken care of.
Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.
Comparison is base (
2f88840) 91.84% compared to head (ae415eb) 91.84%. Report is 5 commits behind head on master.
| Files | Patch % | Lines |
|---|---|---|
| redis/commands/graph/commands.py | 0.00% | 1 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@ Coverage Diff @@
## master #3147 +/- ##
=======================================
Coverage 91.84% 91.84%
=======================================
Files 128 128
Lines 33232 33272 +40
=======================================
+ Hits 30523 30560 +37
- Misses 2709 2712 +3
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
rebased. cc @dvora-h @chayim
Rebased. Looks like @gerzse is on the move here, care to take a look?
Rebased. cc @gerzse.
Rebased. cc @vladvildanov (since you seem to be active on this repo).
cc @petyaslavova
@petyaslavova Conflict fixed. I also moved the configuration to pyproject.toml now that it's a thing, and made it more granular. Some of the non-free Redis modules' command modules don't adhere to usual Python naming, so they're excluded from the naming convention checks.