Skip to content

Commit 9499f42

Browse files
Merge pull request #176 from MikeMcC399/support-eslint-7-and-8
feat: support eslint v7 & v8 instead of ">= 3.2.1"
2 parents 455dc13 + bafb3b6 commit 9499f42

File tree

4 files changed

+14
-32
lines changed

4 files changed

+14
-32
lines changed

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ Note: If you installed ESLint globally then you must also install `eslint-plugin
66

77
## Installation
88

9+
Prerequisites: [ESLint](https://www.npmjs.com/package/eslint) `v7` or `v8`. ESLint `v9` is **not** supported yet.
10+
911
```sh
1012
npm install eslint-plugin-cypress --save-dev
1113
```
@@ -115,8 +117,6 @@ These rules enforce some of the [best practices recommended for using Cypress](h
115117

116118
Rules with a check mark (✅) are enabled by default while using the `plugin:cypress/recommended` config.
117119

118-
**NOTE**: These rules currently require eslint 5.0 or greater. If you would like support added for eslint 4.x, please 👍 [this issue](https://github.com/cypress-io/eslint-plugin-cypress/issues/14).
119-
120120
| | Rule ID | Description |
121121
| :-- | :------------------------------------------------------------------------- | :-------------------------------------------------------------- |
122122
|| [no-assigning-return-values](./docs/rules/no-assigning-return-values.md) | Prevent assigning return values of cy calls |

circle.yml

Lines changed: 10 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,13 @@ workflows:
55
main:
66
jobs:
77
- lint
8-
- test-v5
9-
- test-v6
108
- test-v7
9+
- test-v8
1110
- release:
1211
requires:
1312
- lint
14-
- test-v5
15-
- test-v6
1613
- test-v7
14+
- test-v8
1715
filters:
1816
branches:
1917
only:
@@ -32,22 +30,7 @@ jobs:
3230
name: Lint code
3331
command: npm run lint
3432

35-
test-v5:
36-
docker:
37-
- image: cimg/node:20.12.2
38-
steps:
39-
- checkout
40-
- run:
41-
name: Install dependencies
42-
command: npm ci
43-
- run:
44-
name: Install ESLint 5
45-
command: npm install eslint@5
46-
- run:
47-
name: Test ESLint 5
48-
command: npm test
49-
50-
test-v6:
33+
test-v7:
5134
docker:
5235
- image: cimg/node:20.12.2
5336
steps:
@@ -56,13 +39,13 @@ jobs:
5639
name: Install dependencies
5740
command: npm ci
5841
- run:
59-
name: Install ESLint 6
60-
command: npm install eslint@6
42+
name: Install ESLint 7
43+
command: npm install eslint@7
6144
- run:
62-
name: Test ESLint 6
45+
name: Test ESLint 7
6346
command: npm test
6447

65-
test-v7:
48+
test-v8:
6649
docker:
6750
- image: cimg/node:20.12.2
6851
steps:
@@ -71,10 +54,10 @@ jobs:
7154
name: Install dependencies
7255
command: npm ci
7356
- run:
74-
name: Install ESLint 7
75-
command: npm install eslint@7
57+
name: Install ESLint 8
58+
command: npm install eslint@8
7659
- run:
77-
name: Test ESLint 7
60+
name: Test ESLint 8
7861
command: npm test
7962

8063
release:

package-lock.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
},
2020
"homepage": "https://github.com/cypress-io/eslint-plugin-cypress#readme",
2121
"peerDependencies": {
22-
"eslint": ">= 3.2.1"
22+
"eslint": ">=7 <9"
2323
},
2424
"dependencies": {
2525
"globals": "^13.20.0"
@@ -44,7 +44,6 @@
4444
"semantic-release": "semantic-release",
4545
"start": "yarn run test-watch",
4646
"test": "jest",
47-
"test:v6": "npm i [email protected] && npm run test",
4847
"test-watch": "jest --watchAll",
4948
"prepare": "husky install"
5049
}

0 commit comments

Comments
 (0)