Skip to content

Add ruff lint and format to pre-commit#229

Merged
jbampton merged 1 commit into
john-bampton:mainfrom
jbampton:add-ruff
Apr 20, 2026
Merged

Add ruff lint and format to pre-commit#229
jbampton merged 1 commit into
john-bampton:mainfrom
jbampton:add-ruff

Conversation

@jbampton

Copy link
Copy Markdown
Member

No description provided.

@jbampton jbampton requested a review from BaseMax as a code owner April 20, 2026 21:25
@jbampton jbampton self-assigned this Apr 20, 2026
@jbampton jbampton added this to Turbo Apr 20, 2026
@jbampton jbampton added the tests label Apr 20, 2026
@deepsource-io

deepsource-io Bot commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

DeepSource Code Review

We reviewed changes in 8d5c277...4c08550 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Secrets Apr 20, 2026 9:26p.m. Review ↗
Python Apr 20, 2026 9:26p.m. Review ↗
JavaScript Apr 20, 2026 9:26p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@jbampton jbampton moved this to In Progress in Turbo Apr 20, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request integrates Ruff into the pre-commit workflow and project configuration for linting and formatting. Feedback indicates that the specified Ruff version (v0.15.11) is invalid and should be updated to a stable release. Additionally, it is recommended to consolidate the toolchain by replacing Black and isort with Ruff's native capabilities, which involves enabling the import sorting rule in the configuration to avoid redundancy.

Comment thread .pre-commit-config.yaml
Comment thread .pre-commit-config.yaml
Comment thread pyproject.toml
@jbampton jbampton merged commit 1ab8361 into john-bampton:main Apr 20, 2026
5 of 6 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Turbo Apr 20, 2026
@jbampton jbampton deleted the add-ruff branch April 20, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant