This PR introduces and fixes #392 for automated linting to our Continuous Integration (CI) pipeline. It aims to ensure consistent code style and catch potential errors early in the development process.
Changes
Added a new GitHub Actions workflow file: .github/workflows/ci.yml
Configured the workflow to run on push to main/master branches and on pull requests
Added a .prettierignore to ignore some package managers
Implemented Prettier and ESLint checks for all files
Added specific checks for changed files in pull requests
Implementation Details
The new CI workflow does the following:
Check out the code
Sets up Node.js
Installs dependencies
Runs Prettier on all files
Runs ESLint on all files
For pull requests, run Prettier and ESLint on changed files only
Notes
Caching of npm artifacts was considered but not implemented. We can add this later if build times become an issue.
The workflow is configured to use Node.js version 18. We may need to update this in the future.
Please review and let me know if any changes or additions are needed.
Overview
This PR introduces and fixes #392 for automated linting to our Continuous Integration (CI) pipeline. It aims to ensure consistent code style and catch potential errors early in the development process.
Changes
.github/workflows/ci.yml
.prettierignore
to ignore some package managersImplementation Details
The new CI workflow does the following:
Notes
Please review and let me know if any changes or additions are needed.