Skip to content

Conversation

kingbuzzman
Copy link
Member

Adds all the ruff rules.

Added all the issues under TODO to fix in another PR

@kingbuzzman kingbuzzman requested a review from Copilot August 14, 2025 11:35
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR significantly expands the Ruff linting configuration by adding comprehensive rule sets covering security, performance, documentation, and code quality checks. The purpose is to enforce better code standards across the project while acknowledging that many existing violations will need to be addressed in future PRs.

  • Replaces selective rule enabling (extend-select) with comprehensive rule coverage (select) including 40+ rule categories
  • Adds extensive ignore lists for both general code and test files to handle existing violations
  • Introduces per-file ignore patterns for test directories to allow more lenient standards in test code

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@kingbuzzman kingbuzzman marked this pull request as ready for review August 14, 2025 11:45
@kingbuzzman kingbuzzman enabled auto-merge (squash) August 14, 2025 11:45
@kingbuzzman kingbuzzman merged commit 7e2542a into main Aug 14, 2025
21 checks passed
@kingbuzzman kingbuzzman deleted the dev/ruff-rules branch August 14, 2025 11:46
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