Need for static code analysis setup
Created by: vinay-thallam
It is observed that, though eslint config is present in the repo, source code has many lint rule violations. Eg: assignments to variables that are never used, Usage of == instead of === in few places, missing trailing commas, indentation and space issues.
Not having having proper check on these may result in potential bugs.
This can be solved by adding few things listed below,
- prettier setup for code formatting.
- VS Code recommended extensions file and settings to do format on file save.
- Pre-commit hook to do lint fix and format fix ( takes care if developers not using VS code as their IDE )
- Lint and prettier Validate npm scripts in package.json ( can be invoked manually OR in CI if required)