-
Notifications
You must be signed in to change notification settings - Fork 28
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
Implement Code Linters #86
Comments
Considering https://github.com/super-linter/super-linter as it is the easiest to set up, but lacks a local version for testing changes before committing. |
Hey @PiyushRaj927 couple of things
|
Hey @ChakshuGautam, Regarding the development environment, I appreciate you mentioning pre-commit hooks. I wasn't familiar with them before, but they seem to be exactly what I was looking for. Additionally, incorporating a linter workflow along with the pre-hooks to validate pull requests should fulfill the requirements I had in mind. Regarding my development environment, I'm using Windows 10 with WSL2 Ubuntu and Docker Desktop. |
Hey @PiyushRaj927 can you please twell why #103 was closed, is the |
In order to maintain consistent code quality and identify problems early on, we should incorporate a linters into our workflow. This will assist in streamlining the code review process as well.
The text was updated successfully, but these errors were encountered: