Skip to content

Apply Ruff formatting across source and test files#30

Merged
khanahmedm merged 3 commits intodevelopfrom
feature/ruff-format
Dec 17, 2025
Merged

Apply Ruff formatting across source and test files#30
khanahmedm merged 3 commits intodevelopfrom
feature/ruff-format

Conversation

@khanahmedm
Copy link
Copy Markdown
Collaborator

This PR applies Ruff’s code formatter across the codebase to bring all source and test files into alignment with the enforced formatting standards.

Copy link
Copy Markdown
Collaborator

@Tianhao-Gu Tianhao-Gu left a comment

Choose a reason for hiding this comment

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

can we also add the GHA that runs ruff check and ruff format --check together with this PR? They should pass now.

@@ -0,0 +1,230 @@
"""
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

we want to checkin this file as well?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for catching this folder. It accidently got added. I've removed it now.

I've added .gitignore to this PR.

For GHA, I have another PR #29

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@khanahmedm khanahmedm merged commit d8155db into develop Dec 17, 2025
9 of 11 checks passed
@khanahmedm khanahmedm deleted the feature/ruff-format branch December 17, 2025 17:13
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