Skip to content

chore(deps): upgrade eslint to v10 and fix lint errors - #611

Merged
DerekRoberts merged 1 commit into
mainfrom
feat/replace-pr608
Jul 16, 2026
Merged

chore(deps): upgrade eslint to v10 and fix lint errors#611
DerekRoberts merged 1 commit into
mainfrom
feat/replace-pr608

Conversation

@DerekRoberts

@DerekRoberts DerekRoberts commented Jul 16, 2026

Copy link
Copy Markdown
Member

This PR replaces #608. It upgrades eslint and @eslint/js to v10, configures proper ESLint ignores for Cypress and build config files, and fixes existing lint errors (including a missing toast import in EditForm.jsx).

Supercedes #608


Thanks for the PR!

Any successful deployments (not always required) will be available below.
API available
Frontend available

Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

Copilot AI review requested due to automatic review settings July 16, 2026 19:45
@DerekRoberts
DerekRoberts requested a review from mishraomp as a code owner July 16, 2026 19:45

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR upgrades the frontend linting toolchain to ESLint v10, updates the flat ESLint config ignore patterns, and applies small code cleanups to satisfy the new lint rules (including fixing a missing toast import).

Changes:

  • Bump eslint and @eslint/js to v10 (plus lockfile refresh) and add an overrides entry for eslint-plugin-react.
  • Update frontend/eslint.config.js ignores to exclude build output and Cypress/config files.
  • Fix/clean up frontend component lint issues (unused React imports, hook deps, and missing toast import).

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
frontend/src/components/NotFound.jsx Removes unused React import and fixes unescaped apostrophe in JSX text.
frontend/src/components/LeftDrawer.jsx Removes unused icon and useState imports.
frontend/src/components/FormComponent.jsx Removes unused React default import/unused state setter and adjusts useEffect deps.
frontend/src/components/EditForm.jsx Removes unused React imports/state setter and adds missing toast import.
frontend/package.json Upgrades ESLint deps to v10 and adds an override to align peer dependency resolution.
frontend/package-lock.json Updates lockfile for ESLint v10 dependency tree.
frontend/eslint.config.js Expands ignore globs for dist/Cypress/config files.
frontend/env.js Removes now-unnecessary eslint-disable comment.
Files not reviewed (1)
  • frontend/package-lock.json: Generated file

@DerekRoberts DerekRoberts self-assigned this Jul 16, 2026
@DerekRoberts
DerekRoberts merged commit 1e10ae1 into main Jul 16, 2026
13 checks passed
@DerekRoberts
DerekRoberts deleted the feat/replace-pr608 branch July 16, 2026 20:05
@github-project-automation github-project-automation Bot moved this from New to Done in DevOps (NR) Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants