fix: clean unused eslint disables

[MikeMcC399]

Issue

ESLint 9.x identifies usages in the repo of unnecessary ESLint rule disables.

This affects the following instances:

/* eslint-disable-next-line no-console */ in

• cypress/e2e/2-advanced-examples/connectors.cy.js
• cypress/e2e/2-advanced-examples/cypress_api.cy.js
• cypress/e2e/2-advanced-examples/spies_stubs_clocks.cy.js

The ESLint no-console rule does not belong to the set of recommended rules in ESLint 8.x or 9.x and so disabling it in-line was not necessary.

/* eslint-disable no-unused-vars */ in

• cypress/e2e/2-advanced-examples/navigation.cy.js

ESLint finds no unused variables in the marked section.

Process

By temporarily updating to ESLint 9.x, unnecessary rule disables are identified and automatically removed by eslint --fix

Run ESLint 9.x:

npm ci
npm install eslint@9 --force
npm run lint

Revert to ESLint 8.x and check that linting still works:

git restore package.json package-lock.json
npm ci
npm run lint

Changes

• Automatic fixes from ESLint are applied.
• Additionally associated left-over blank lines are removed.

This prepares the Cypress test specs so that they will pass linting when the repo is migrated to ESLint 9.x.

Verification

On Ubuntu 22.04.4 LTS, Node.js v20.12.2 LTS

npm ci
npm run lint
npm run local:run

[cypress-app-bot]

• Create a Draft Pull Request if your PR is not ready for review. Mark the PR as Ready for Review when you're ready for a Cypress team member to review the PR.

[cypress-app-bot]

🎉 This PR is included in version 2.0.7 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀