Skip to content

chore: remove stale setup.cfg flake8 config - #8

Merged
elsisiem merged 1 commit into
mainfrom
chore/remove-stale-flake8
Oct 27, 2026
Merged

elsisiem merged 1 commit into
mainfrom
chore/remove-stale-flake8

Conversation

@elsisiem

@elsisiem elsisiem commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

setup.cfg held only a flake8 section. Linting runs through ruff, configured in pyproject.toml with the same line length, and CI does not run flake8.

setup.cfg only held a flake8 section. Linting runs through ruff, which is
configured in pyproject.toml with the same 140 character line length, and
CI does not run flake8.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 7 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2e05c646-1194-4ce9-a41f-64a8962e40f9
Commits

Reviewing files that changed from the base of the PR and between 36f5c5b and ac12f00.

Files selected for processing: 1
  • setup.cfg
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Remove unused flake8 configuration

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Delete the flake8-only setup.cfg because CI runs Ruff instead.
• Keep lint settings in pyproject.toml, where Ruff already uses the same 140-character line
 length.
High-Level Assessment

Deleting the unused file is the simplest approach. CI runs Ruff, and its configuration already specifies the same line length; retaining a separate flake8 configuration would add no value.

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can describe a rule in plain language on the Rules page and Qodo drafts it for you

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@elsisiem
elsisiem merged commit 6f72ec0 into main Oct 27, 2026
3 checks passed
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.

1 participant