clean-react-app icon indicating copy to clipboard operation
clean-react-app copied to clipboard

[FEATURE] Add ESLint and Implement It in the Project

Open rubemfsv opened this issue 2 years ago • 8 comments

Description: We are considering the implementation of ESLint in our project. ESLint is a widely used JavaScript linter that helps identify and enforce coding standards, detect potential issues, and maintain code quality. This feature request aims to add ESLint to our development workflow and ensure that our codebase adheres to best practices and coding standards.

Expected Behavior: Upon implementing ESLint, we expect the following behavior:

  1. Code Quality Checks: ESLint should analyze our JavaScript code for potential errors, style violations, and best practice adherence.
  2. Customizable Rules: We should configure ESLint rules to match our preferred coding standards and conventions.
  3. Automatic Code Checks: ESLint should be integrated into our development process to automatically check code as it is written or during CI/CD pipeline checks (we have Husky, you could put there in this first moment).

Screenshots (if applicable): N/A

Additional Information: Adding ESLint to our project can significantly improve code quality, maintainability, and consistency. It helps catch potential issues early in the development process and enforces coding standards, making our codebase more robust and easier to maintain.

rubemfsv avatar Oct 02 '23 13:10 rubemfsv

I would like to work on this

dhaneshragu avatar Oct 02 '23 16:10 dhaneshragu

@dhaneshragu sure! Don't forget to ensure commit messages are following the conventional commits 😄

rubemfsv avatar Oct 02 '23 17:10 rubemfsv

@rubemfsv pls check the PR #57

dhaneshragu avatar Oct 04 '23 10:10 dhaneshragu

@dhaneshragu I've put some points in the PR

rubemfsv avatar Oct 08 '23 13:10 rubemfsv

@dhaneshragu are you going to fix or I can assign to other people?

rubemfsv avatar Oct 12 '23 12:10 rubemfsv

@dhaneshragu are you going to fix or I can assign to other people?

Sorry I got busy with my university, You can assign it to other people

dhaneshragu avatar Oct 12 '23 12:10 dhaneshragu

I would like to take this one @rubemfsv

rockingrohit9639 avatar Oct 27 '23 17:10 rockingrohit9639

Sure, @rockingrohit9639!

rubemfsv avatar Oct 27 '23 17:10 rubemfsv