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

Add support for Ruff #90

Open
fazeelghafoor opened this issue Jul 29, 2024 · 0 comments
Open

Add support for Ruff #90

fazeelghafoor opened this issue Jul 29, 2024 · 0 comments

Comments

@fazeelghafoor
Copy link

fazeelghafoor commented Jul 29, 2024

I have a proposal for adding ruff for this project to improve code quality and standard according to pep8. I have forked the repo and checked that it is already using black and pyre but I have a suggestion to use ruff instead for both as a linter and formatter instead of black.

.pre-commit-config.yaml:

    rev: "v0.5.4"
    hooks:
      - id: ruff
        args: ["--fix"]
      - id: ruff-format

this will resolve all the unused imports and catch any unused code like the following

Undefined name time RuffF821

"time" is not defined PylancereportUndefinedVariable

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

No branches or pull requests

1 participant