This PR integrates the Prettier and ESLint configuration rules into our project. In addition, it introduces a Prettier check step into our continuous integration (CI) process.
Configurations Adjusted:
Prettier: Following our coding style preferences, I've set Prettier to favour single quotes over double quotes but kept the sem-colon (eww). @davidtaing , I noticed the existing codebase uses double quotes; if you have a strong preference for keeping double quotes, please let me know. We can easily run a global replacement to revert this change.
ESLint: The new ESLint rules are now active. I made several adjustments, such as reordering imports, to satisfy ESLint's requirements. It appears that Next.js already handles linting during the build process. For added assurance, I've included a prettier:check step in the CI pipeline to run after tests complete.
This PR integrates the Prettier and ESLint configuration rules into our project. In addition, it introduces a Prettier check step into our continuous integration (CI) process.
Configurations Adjusted:
prettier:check
step in the CI pipeline to run after tests complete.