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

Outdated peerDependencies eslint >= 3.2.1 #175

Closed
MikeMcC399 opened this issue Apr 18, 2024 · 2 comments · Fixed by #176
Closed

Outdated peerDependencies eslint >= 3.2.1 #175

MikeMcC399 opened this issue Apr 18, 2024 · 2 comments · Fixed by #176

Comments

@MikeMcC399
Copy link
Collaborator

Issue

The eslint-plugin-cypress plugin currently defines peerDependencies of "eslint": ">= 3.2.1". This includes versions of ESLint v3, v4 and v9 which are not compatible with the current latest version of eslint-plugin-cypress@2.15.2 (including its other peerDependencies). ESLint v8 cannot be linted by @cypress/eslint-plugin-dev and replacement linting recommended by Linting a Plugin only covers ESLint v7 and above.

The intersection of supportable combinations is ESLint v7 and v8 only.

In detail:

ESLint v3 and v4

ESLint v5 and v6

ESLint v7

  • "eslint": "^7.32.0" is the currently configured version of ESLint in the master branch of eslint-plugin-cypress. This version lints with ESLint and tests with Jest successfully. This version is already covered by peerDependencies of "eslint": ">= 3.2.1".

ESLint v8

ESLint v9

  • Due to breaking changes in ESLint v9 (see Migrate to v9.x related to removed API calls and flat-file related changes) the plugin eslint-plugin-cypress@2.15.2 (current latest) is not compatible with ESLint v9. peerDependencies of eslint-plugin-cypress needs to be "eslint": "<9".

Suggestion

Considering all of the above issues, eslint-plugin-cypress needs to have its peerDependencies set to the semver range:

"eslint": ">=7 <9"

This is a breaking change needing a new major release eslint-plugin-cypress of 3.x.

Implementing this change has the benefits:

@MikeMcC399
Copy link
Collaborator Author

@cypress-app-bot
Copy link

🎉 This issue has been resolved 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 a pull request may close this issue.

2 participants