Add Ruff Linter Integration and Pre-Commit Hook #205
Conversation
| @@ -0,0 +1,20 @@ | |||
| exclude: ^docs/ | |||
There was a problem hiding this comment.
If I am not wrong, this prevents running pre-commit on any file inside docs dir?
There was a problem hiding this comment.
yeah, we can see if including docs in linters would be beneficial or not, this was done as an experiment we can check if mdx is also parsed by pre-comit.
There was a problem hiding this comment.
okay, let me have a look.
| - repo: https://github.com/pre-commit/pre-commit-hooks | ||
| rev: v4.5.0 | ||
| hooks: | ||
| - id: check-added-large-files |
There was a problem hiding this comment.
any specific reason for checking for large files? 🤔 I don't think thats required.
Also, adding requirements-txt-fixer and check-yaml might be helpful.
There was a problem hiding this comment.
This is a template that was taken from another project that I worked on to check the feasibility of including it in this project, you can make any modifications that you see fit.
Overview
This PR implements the integration of the Ruff linter into our Python project and sets up a pre-commit hook to enforce code quality standards before commits are made.