Add GitHub Actions workflow to enforce ESLint on PRs

[Copilot]

Adds automated linting checks that block PR merges when code style violations are detected.

Changes

• .github/workflows/lint.yml: New workflow that runs linting on all PRs
  • Uses Node.js 20 with npm caching
  • Explicit contents: read permissions per security best practices
  • Runs npm run lint to check for linting errors
  • Fails workflow on non-zero exit from ESLint

Behavior

The workflow runs automatically on PR creation and updates. It will:

1. Check code for linting errors using ESLint
2. Block merge via required status checks if linting errors are found (requires branch protection configuration in repository settings)

The workflow does not attempt to automatically fix linting issues - developers must fix any reported errors manually before the PR can be merged.

Original prompt

create a workflow which runs on PRs which checks linting and blocks merging if linting fails

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[JohnMarsden24]

@copilot check the workflow, it's failing I think we need to add a command to fix linting issues

[Copilot]

@copilot check the workflow, it's failing I think we need to add a command to fix linting issues

I've updated the workflow to automatically fix linting issues that can be auto-fixed. The changes include:

1. Added lint:fix script to package.json (commit 1754b08)
2. Updated workflow to run npm run lint:fix before the final check (commit 1754b08)
3. The workflow now commits and pushes any auto-fixes (commit 28788ec)
4. Final lint check still runs to ensure all issues are resolved

The workflow will now:

• Attempt to auto-fix fixable issues
• Commit and push those fixes automatically
• Still fail if there are non-fixable linting errors (like the current react-refresh/only-export-components error in CartContext.tsx)

You can also manually run npm run lint:fix locally to fix issues before committing.

[JohnMarsden24]

@copilot the lint fix script was added in another PR so rebase and remove it from this PR commit. We also just want to check linting in the workflow, not fix it, go back to the original commit

[Copilot]

@copilot the lint fix script was added in another PR so rebase and remove it from this PR commit. We also just want to check linting in the workflow, not fix it, go back to the original commit

Reverted the workflow and package.json to the original simple linting check (commit 6a3118e). The changes:

• Removed lint:fix script from package.json
• Reverted workflow to just run npm run lint without auto-fixing
• Changed permissions back to contents: read
• Removed all auto-fix and commit/push logic

The workflow now only checks linting and blocks merging if it fails.