Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed ruff I001 rule #10272

Closed
wants to merge 2 commits into from

Conversation

ananyakaligal
Copy link
Contributor

Closes #10261

Description
This pull request resolves I001 by sorting and organizing import statements across the codebase, enhancing readability and maintainability.

Technical Details

  • Complies with Ruff's I001 rule for unsorted imports.
  • Standardizes import formatting for improved clarity.

Testing

  • Linting completed with no errors (I001 resolved).
  • All tests passed successfully.

Stakeholders
@RayBB

image

@RayBB
Copy link
Collaborator

RayBB commented Jan 4, 2025

It looks like precommit has reverted you changes. This is probably because you have an outdated version of rust. Try running ruff --version and compare to the version in the requirements.

@ananyakaligal
Copy link
Contributor Author

ananyakaligal commented Jan 4, 2025

@RayBB My ruff is up to date with the version in the requirements.
image

@RayBB
Copy link
Collaborator

RayBB commented Jan 4, 2025

I'm closing this now since we see the fix isn't to run ruff and will comemnt on the issue so it's easier for people to track if they run into this issue.

@RayBB RayBB closed this Jan 4, 2025
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.

Unsorted and Unformatted Import Blocks in Multiple Files
2 participants