Skip to content

Commit b273fe5

Browse files
MichaelDeBoeyljharb
authored andcommitted
[New] Support ESLint 7.x
1 parent 1a97632 commit b273fe5

11 files changed

+10
-24
lines changed

.travis.yml

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
11
language: node_js
22
node_js:
3+
- "14"
34
- "12"
45
- "10"
5-
- "9"
66
- "8"
7-
- "7"
87
- "6"
9-
- "5"
108
- "4"
119
cache:
1210
yarn: true
@@ -29,6 +27,7 @@ env:
2927
global:
3028
- TEST=true
3129
matrix:
30+
- ESLINT=7
3231
- ESLINT=6
3332
- ESLINT=5
3433
- ESLINT=4
@@ -45,19 +44,15 @@ matrix:
4544
env: ESLINT=5
4645
- node_js: "4"
4746
env: ESLINT=6
48-
- node_js: "5"
49-
env: ESLINT=5
47+
- node_js: "4"
48+
env: ESLINT=7
5049
- node_js: "6"
5150
env: ESLINT=6
51+
- node_js: "6"
52+
env: ESLINT=7
53+
- node_js: "8"
54+
env: ESLINT=7
5255
allow_failures:
53-
- node_js: "7"
54-
env: ESLINT=6
55-
- node_js: "5"
56-
env: ESLINT=6
57-
- node_js: "5"
58-
env: ESLINT=4
59-
- node_js: "5"
60-
env: ESLINT=3
6156
- node_js: "4"
6257
env: ESLINT=4
6358
- node_js: "4"

__tests__/src/rules/alt-text-test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ const array = [{
5050
'input[type="image"]': ['InputImage'],
5151
}];
5252

53-
5453
ruleTester.run('alt-text', rule, {
5554
valid: [
5655
// DEFAULT ELEMENT 'img' TESTS

__tests__/src/rules/label-has-for-test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ const expectedEveryError = {
3434
type: 'JSXOpeningElement',
3535
};
3636

37-
3837
const optionsComponents = [{
3938
components: ['Label', 'Descriptor'],
4039
}];

__tests__/src/rules/no-onchange-test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
* @author Ethan Cohen
55
*/
66

7-
87
// -----------------------------------------------------------------------------
98
// Requirements
109
// -----------------------------------------------------------------------------

__tests__/src/rules/role-has-required-aria-props-test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ const errorMessage = (role) => {
2929
};
3030
};
3131

32-
3332
// Create basic test cases using all valid role types.
3433
const basicValidityTests = [...roles.keys()].map((role) => {
3534
const {

__tests__/src/util/hasAccessibleChild-test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ describe('hasAccessibleChild', () => {
1111
});
1212
});
1313

14-
1514
describe('has no children and sets dangerouslySetInnerHTML', () => {
1615
it('Returns true', () => {
1716
const prop = JSXAttributeMock('dangerouslySetInnerHTML', true);

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
"babel-jest": "^24.9.0",
3838
"babel-preset-airbnb": "^5.0.0",
3939
"coveralls": "^3.1.0",
40-
"eslint": "^3 || ^4 || ^5 || ^6",
40+
"eslint": "^3 || ^4 || ^5 || ^6 || ^7",
4141
"eslint-config-airbnb-base": "^14.2.0",
4242
"eslint-plugin-flowtype": "^5.1.3",
4343
"eslint-plugin-import": "^2.21.2",
@@ -70,7 +70,7 @@
7070
"jsx-ast-utils": "^2.4.1"
7171
},
7272
"peerDependencies": {
73-
"eslint": "^3 || ^4 || ^5 || ^6"
73+
"eslint": "^3 || ^4 || ^5 || ^6 || ^7"
7474
},
7575
"jest": {
7676
"coverageReporters": [

src/rules/alt-text.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,6 @@ module.exports = {
218218
return type;
219219
}));
220220

221-
222221
return {
223222
JSXOpeningElement: (node) => {
224223
const nodeType = elementType(node);

src/rules/anchor-has-content.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import { elementType } from 'jsx-ast-utils';
1111
import { arraySchema, generateObjSchema } from '../util/schemas';
1212
import hasAccessibleChild from '../util/hasAccessibleChild';
1313

14-
1514
const errorMessage = 'Anchors must have content and the content must be accessible by a screen reader.';
1615

1716
const schema = generateObjSchema({ components: arraySchema });

src/rules/no-distracting-elements.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ const errorMessage = (element) => (
1414
`Do not use <${element}> elements as they can create visual accessibility issues and are deprecated.`
1515
);
1616

17-
1817
const DEFAULT_ELEMENTS = [
1918
'marquee',
2019
'blink',

src/rules/no-noninteractive-tabindex.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ module.exports = {
5858
}
5959
const role = getLiteralPropValue(getProp(node.attributes, 'role'));
6060

61-
6261
if (!dom.has(type)) {
6362
// Do not test higher level JSX components, as we do not know what
6463
// low-level DOM element this maps to.

0 commit comments

Comments
 (0)