Skip to content

Conversation

@MikeMcC399
Copy link
Collaborator

Situation

scripts/set-port.js includes the ESLint configuration comment:

/* eslint-env node */

This is no longer used in ESLint flat configurations, and will cause an error in ESLint 10.x.

See Configuration Migration Guide eslint-env Configuration Comments which says:

... when migrating from eslintrc to flat config, eslint-env configuration comments should be removed from all files.

This repo is already using an ESLint 9.x flat configuration.

Change

Remove the unused eslint-env configuration comment from scripts/set-port.js.

Verification

npm ci
npx eslint

Confirm no errors reported.

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399 MikeMcC399 self-assigned this Nov 17, 2025
@MikeMcC399 MikeMcC399 force-pushed the remove/eslint-env-comments branch from e2a36da to 416726a Compare November 17, 2025 08:42
@MikeMcC399 MikeMcC399 marked this pull request as ready for review November 17, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants