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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: drop testing of ESLint 4 #165
test: drop testing of ESLint 4 #165
Conversation
|
@MikeMcC399 I removed the v4 job as a required job for branch protections |
cd8120b
to
4811303
Compare
Thank you very much! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MikeMcC399 Looks good. Thanks for the detailed write up
馃帀 This PR is included in version 3.0.0 馃帀 The release is available on: Your semantic-release bot 馃摝馃殌 |
related to issues:
npm ci
on repo source聽#153Issue
@cypress/eslint-plugin-dev@3.2.0 configured in this repo's
devDependencies
containspeerDependencies
:however
@typescript-eslint/parser@1.11.0 is only compatible with ESLint
5
:This rules out the supported use of ESLint
4
for dev linting tests in the repository.Change
Remove tests for ESLint
4
.The last release of ESLint 4.x was eslint@4.19.1 released on March 22, 2018, more than 6 years ago. The current release is eslint@9.0.0 released on Apr 5, 2024.
This change does not prevent
eslint-plugin-cypress
being installed and used with ESLint4
therefore it is not a breaking change for end-users.There is an outdated note in README > Rules referring to ESLint
4.x
. The issue #14, being referred to, is however already resolved. This should be cleaned up in a later migration step as the repo is moved towards supporting ESLint8.x
correctly.