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

Added a better pre commit (S.R No. 10) #352

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Rishab87
Copy link
Contributor

@Rishab87 Rishab87 commented Feb 20, 2025

I've added a better pre commit for the whole project which on commit automatically checks staged files and fixes them.
For context earlier the project was unformatted and had a bad linter just in frontend which gave uncessary errors
Added a workflow for it too!
Before committing make sure your project is setup properly
Fixes #331

@Rishab87 Rishab87 marked this pull request as draft February 20, 2025 07:43
@priyankeshh
Copy link

Just wanted to let you know there is already a SR NO. 9 #336, make this PR SR 10

@Rishab87 Rishab87 changed the title Added a better pre commit (S.R No. 9) Added a better pre commit (S.R No. 10) Feb 20, 2025
@Rishab87
Copy link
Contributor Author

@priyankeshh thanks for letting me know

@Rishab87 Rishab87 marked this pull request as ready for review February 20, 2025 14:32
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.

Feat:Better pre commit hook for linting
2 participants