Skip to content

feat: Add Ruff linting workflow and fix code quality issues#465

Open
neverbiasu wants to merge 3 commits intoOpen-Deep-ML:mainfrom
neverbiasu:ruff
Open

feat: Add Ruff linting workflow and fix code quality issues#465
neverbiasu wants to merge 3 commits intoOpen-Deep-ML:mainfrom
neverbiasu:ruff

Conversation

@neverbiasu
Copy link

Summary

Added automated Python linting with Ruff and fixed existing code quality issues across the codebase.

Changes

  • Added GitHub Actions workflow for Ruff linting
  • Fixed 25+ linting issues (imports, unused variables, formatting, etc.)
  • Excluded problem.py files (educational templates)
  • All existing functionality preserved

Benefits

  • Consistent code style
  • Automated quality checks for future PRs
  • Better maintainability

Files modified: 10+ solution files across various problems

Workflow Results

  1. Ruff detected code quality issues and action failed.
    image

  2. Action successed.
    image

@neverbiasu neverbiasu marked this pull request as ready for review June 7, 2025 13:45
@neverbiasu
Copy link
Author

@moe18 please review my PR, any suggestions are appreciated.

@moe18
Copy link
Collaborator

moe18 commented Jun 15, 2025

@neverbiasu thank you for the work you put into this, would you be able to make it work with the new structure of the repo, if not I will work on it

@neverbiasu
Copy link
Author

@moe18 no problem, I'll work on it

@neverbiasu
Copy link
Author

@moe18 I've revised, please check if its meet your requires.

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

Comments