Add Linting as a part of CI pipeline

tarunsinghofficial commented 3 weeks ago

Overview

We should add automated linting to our Continuous Integration (CI) pipeline to ensure consistent code style and catch potential errors early in the development process.

Current Situation

Currently, there is no automated linting in our CI pipeline.
Code style inconsistencies are being caught manually during code reviews.
We have .eslintrc.json and .prettierrc.json files in our repository, but they are not enforced automatically.

Proposed Solution

Add a linting step to our GitHub Actions workflow that runs on every pull request and push to main branches.

Implementation Details

Create or update a GitHub Actions workflow file (e.g., .github/workflows/ci.yml).
Add a linting job that includes:
- Setting up Node.js
- Installing dependencies
- Running ESLint
- (Optional) Running Prettier

tarunsinghofficial commented 3 weeks ago

I'm interested in doing this issue. Let me know if it is good, and please assign the issue to me. Thanks!

hoch commented 3 weeks ago

Sure thing. I'll assign this to you.

tarunsinghofficial commented 1 week ago

Sure thing. I'll assign this to you.

Hi @hoch

Please review PR #393, whenever you're free. Thanks :)

hoch commented 1 week ago

Yes. We will get to this soon after wrapping up the https://github.com/GoogleChromeLabs/web-audio-samples/pull/395. Thanks for your contribution!

GoogleChromeLabs / web-audio-samples