Skip to content

Conversation

@FarhanAnjum-opti
Copy link
Contributor

@FarhanAnjum-opti FarhanAnjum-opti commented Feb 9, 2026

Summary

  • Replace Flake8 with Ruff as the primary Python linter
  • Update lint configuration and CI to use Ruff

The change simplifies the linting setup, improves performance, and reduces dependency overhead while maintaining equivalent linting coverage. Linting errors that do not break code logic can be auto fixed using
ruff check . --fix

Test plan

  • Run ruff check . locally
  • Run full CI pipeline and verify lint and test steps pass

Issues

FSSDK-12276

Copy link
Contributor

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

Replaces Flake8 with Ruff as the project’s Python linter by adding Ruff configuration, updating developer docs, and switching CI to run ruff check.

Changes:

  • Add ruff.toml with Flake8-equivalent rule selection (E/W/F) and project excludes.
  • Swap Flake8 for Ruff in test requirements and GitHub Actions lint job.
  • Update contributor documentation to reference Ruff; remove legacy .flake8.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
ruff.toml Introduces Ruff config (line length, excludes, selected rule families, per-file ignores).
requirements/test.txt Replaces flake8 dependency with ruff.
CONTRIBUTING.md Updates contributor workflow and style notes to use Ruff.
.github/workflows/python.yml Updates CI lint job to install/run Ruff instead of Flake8.
.flake8 Removes obsolete Flake8 configuration file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

@Mat001 Mat001 left a comment

Choose a reason for hiding this comment

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

LGTM. Make sure that no errors in PR checks.

@FarhanAnjum-opti FarhanAnjum-opti merged commit 494e18b into master Feb 10, 2026
15 checks passed
@FarhanAnjum-opti FarhanAnjum-opti deleted the farhan/FSSDK-12276-replace-flake8-with-ruff branch February 10, 2026 16:11
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