-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Set up JavaScript and Typescript linting and formatting tools #64
Comments
We already have prettier configured https://github.com/SasanLabs/VulnerableApp-facade/blob/main/facade-app/package.json#L27 If you still think we can make it better or integrate with npm build please go ahead |
Hi @preetkaran20, I'd be happy to work on this. I think there is value in having both ESLint and Prettier, but let me know what you think best suits the need of the project. Given that their rules can conflict with each other, I'll spend some time reviewing their configs and defaults to minimize that. I would be happy to set up a husky hook to run ESLint and/or Prettier on commit too, so that devs don't have to run the extra prettier command before committing. |
@kkreine please go ahead with the task. cc: @Dripcoding |
I'll be working on this this weekend. |
Hi @preetkaran20, @Dripcoding, sorry for being a little late on this, but I have Husky and ESLint configured and working with Prettier. However, there are a few default ESLint rules the code is breaking, including:
I can send over more detail & files affected if you'd like. Would you like me to address these issues? I'd be happy to, and many are pretty straightforward, but I wanted to check before I go beyond the initial scope of this issue. I could also disable those rules globally or locally. |
@kkreine thanks for pointing those issues, please go ahead with fixing those. thanks, |
@kkreine are you still working on this ? |
Is your feature request related to a problem? Please describe.
I would like to automate linting and formatting when contributing code. This will make collaboration easier and helps ensure code-quality.
Describe the solution you'd like
I would like ESLint, TSLint, Husky, Prettier to be configured for the project locally and as part of the ci/cd pipeline.
Describe alternatives you've considered
I've also considered looking in to using code quality tools available on the Github Actions marketplace.
Additional context
NA
Would you like to help fix this issue?
Yes, I'd like to work on this feature.
The text was updated successfully, but these errors were encountered: