Skip to content

Conversation

@pre-commit-ci
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Sep 22, 2025

@pre-commit-ci pre-commit-ci bot requested a review from Devasy23 as a code owner September 22, 2025 21:56
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 22, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Sep 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.42%. Comparing base (a035ec0) to head (e6a7b31).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #174   +/-   ##
=======================================
  Coverage   74.42%   74.42%           
=======================================
  Files          17       17           
  Lines        1693     1693           
  Branches      158      158           
=======================================
  Hits         1260     1260           
  Misses        381      381           
  Partials       52       52           
Components Coverage Δ
Authentication System 75.45% <ø> (ø)
Expense Management 70.64% <ø> (ø)
Group Management 76.20% <ø> (ø)
User Management 97.16% <ø> (ø)
Backend Core 69.41% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 2873ed3 to 51d187e Compare October 13, 2025 21:52
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 51d187e to 6efe3ab Compare November 10, 2025 22:19
@Devasy23
Copy link
Owner

@coderabbitai review that if we can skip isort and black and flake8 just by using ruff

@netlify
Copy link

netlify bot commented Nov 27, 2025

Deploy Preview for split-but-wiser canceled.

Name Link
🔨 Latest commit e6a7b31
🔍 Latest deploy log https://app.netlify.com/projects/split-but-wiser/deploys/69374e0453dfdd0008eb2a09

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 27, 2025

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check the pre-commit configuration
cat .pre-commit-config.yaml

Length of output: 358


🏁 Script executed:

#!/bin/bash
# Check for ruff configuration in pyproject.toml
if [ -f pyproject.toml ]; then
  cat pyproject.toml
fi

Length of output: 0


🏁 Script executed:

#!/bin/bash
# Check for standalone ruff configuration
if [ -f ruff.toml ]; then
  cat ruff.toml
elif [ -f .ruff.toml ]; then
  cat .ruff.toml
fi

Length of output: 0


🏁 Script executed:

#!/bin/bash
# Check what Python files exist in the project to understand the codebase structure
fd -e py | head -20

Length of output: 569


@Devasy23: Yes, you can absolutely replace isort, black, and flake8 with Ruff! Ruff is designed as a drop-in replacement for these tools and offers significant advantages.

Why Ruff?

  1. Single Tool: Replaces all three tools with one
  2. Speed: 10-100x faster than the tools it replaces
  3. Compatibility: ruff format is compatible with Black's formatting, and Ruff includes isort-compatible import sorting
  4. Comprehensive: 800+ linting rules covering flake8 and popular plugins

Proposed .pre-commit-config.yaml Update

Replace your current configuration with:

repos:
  - repo: https://github.com/astral-sh/ruff-pre-commit
    rev: v0.8.4  # Use the latest version
    hooks:
      - id: ruff
        args: [--fix]  # Auto-fix issues where possible
      - id: ruff-format  # Format code (replaces black)

Recommended pyproject.toml Configuration

Create a pyproject.toml file (or add to existing) to configure Ruff:

[tool.ruff]
line-length = 88  # Match Black's default
target-version = "py311"  # Adjust to your Python version

[tool.ruff.lint]
select = [
    "E",   # pycodestyle errors (replaces flake8)
    "F",   # pyflakes (replaces flake8)
    "I",   # isort (replaces isort)
    "W",   # pycodestyle warnings
]

[tool.ruff.lint.isort]
profile = "black"  # Match your current isort config

Migration Steps

  1. Update .pre-commit-config.yaml with the Ruff configuration above
  2. Create pyproject.toml with Ruff settings
  3. Run pre-commit autoupdate to get the latest Ruff version
  4. Test with pre-commit run --all-files

Would you like me to create an issue to track this migration, or would you prefer to implement it in this PR?

updates:
- https://github.com/psf/blackhttps://github.com/psf/black-pre-commit-mirror
- [github.com/psf/black-pre-commit-mirror: 25.1.0 → 25.12.0](psf/black-pre-commit-mirror@25.1.0...25.12.0)
- [github.com/pycqa/isort: 6.0.1 → 7.0.0](PyCQA/isort@6.0.1...7.0.0)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from b8067ff to e6a7b31 Compare December 8, 2025 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants