Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): update eslint to 7.32.0 #172

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

MikeMcC399
Copy link
Collaborator

Issue

The repo is currently based on [email protected] released on Mar 30, 2019. This is an old version released 5 years ago and the repo needs to be moved towards compatibility with the current version ESLint 9.x.

Changes

  1. The repo base version is updated to [email protected] released on Jul 31, 2021. This version of ESLint is used for the Lint test in CircleCI. ESLint 7.x is an intermediate step in the journey towards supporting ESLint 9.x.

  2. The following two modules are updated:

  1. CircleCI Jest tests are extended to cover ESLint v7.

The equivalent npm package installation instructions used are:

npm install eslint@7 -D
npm install @cypress/eslint-plugin-dev@3 @cypress/eslint-plugin-json@3 -D -E

Cross-version testing

On Ubuntu 22.04.4 LTS, with Node.js 20.12.2 and with base version [email protected] installed, cross-version testing shows all tests passing. The symbol "+" indicates a successful test.

ESLint v5 v6 v7
npm run lint + + +
npm test + + +

Add ESLint 7 to CircleCI tests
@cypress-app-bot
Copy link

@MikeMcC399 MikeMcC399 marked this pull request as ready for review April 17, 2024 13:25
Copy link
Member

@jennifer-shehane jennifer-shehane left a comment

Choose a reason for hiding this comment

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

@MikeMcC399 Nice. I think an incremental approach is a great idea.

@jennifer-shehane jennifer-shehane merged commit 72b5124 into cypress-io:master Apr 17, 2024
9 checks passed
@MikeMcC399
Copy link
Collaborator Author

@jennifer-shehane

Nice. I think an incremental approach is a great idea.

Thanks again for reviewing and merging! And thanks also for your vote of confidence in the Collaborator/Triage role!

I did consider a "big-bang" update approach, however there are many complex interdependencies which are difficult to understand in one go. Putting all changes into one big PR would also be very difficult to review in-depth. So I have gone with the incremental approach and I'm happy with this so far! 🙂

@MikeMcC399 MikeMcC399 deleted the update/eslint-to-7 branch April 17, 2024 13:53
@cypress-app-bot
Copy link

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants