Skip to content

Commit cd082f4

Browse files
committed
Merge pull request #527 from airbnb/jake/legacy-ruleset
Create eslint-config-airbnb/legacy to export ES5-only rules
2 parents 16f81fb + b42e9b8 commit cd082f4

File tree

4 files changed

+45
-25
lines changed

4 files changed

+45
-25
lines changed

packages/eslint-config-airbnb/README.md

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,16 +4,30 @@ This package provides Airbnb's .eslintrc as an extensible shared config.
44

55
## Usage
66

7-
### With React Style
7+
We export three ESLint configurations for your usage.
88

9-
1. `npm install --save-dev eslint-config-airbnb babel-eslint eslint-plugin-react`
9+
### eslint-config-airbnb
10+
11+
Our default export contains all of our ESLint rules, including EcmaScript 6+
12+
and React. It requires `eslint`, `babel-eslint`, and `eslint-plugin-react`.
13+
14+
1. `npm install --save-dev eslint-config-airbnb babel-eslint eslint-plugin-react eslint`
1015
2. add `"extends": "airbnb"` to your .eslintrc
1116

12-
### Without React Style
17+
### eslint-config-airbnb/base
18+
19+
Lints ES6+ but does not lint React. Requires `eslint` and `babel-eslint`.
1320

14-
1. `npm install --save-dev eslint-config-airbnb babel-eslint `
21+
1. `npm install --save-dev eslint-config-airbnb babel-eslint eslint`
1522
2. add `"extends": "airbnb/base"` to your .eslintrc
1623

24+
### eslint-config-airbnb/legacy
25+
26+
Lints ES5 and below. Only requires `eslint`.
27+
28+
1. `npm install --save-dev eslint-config-airbnb eslint`
29+
2. add `"extends": "airbnb/legacy"` to your .eslintrc
30+
1731
See [Airbnb's Javascript styleguide](https://github.com/airbnb/javascript) and
1832
the [ESlint config docs](http://eslint.org/docs/user-guide/configuring#extending-configuration-files)
1933
for more information.

packages/eslint-config-airbnb/base.js

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,8 @@
11
module.exports = {
22
'extends': [
3-
'./rules/best-practices.js',
4-
'./rules/errors.js',
5-
'./rules/es6.js',
6-
'./rules/legacy.js',
7-
'./rules/node.js',
8-
'./rules/strict.js',
9-
'./rules/style.js',
10-
'./rules/variables.js'
3+
'eslint-config-airbnb/legacy',
4+
'eslint-config-airbnb/rules/es6',
115
],
126
'parser': 'babel-eslint',
13-
'env': {
14-
'browser': true,
15-
'node': true,
16-
'amd': false,
17-
'mocha': false,
18-
'jasmine': false
19-
},
20-
'ecmaFeatures': {},
21-
'globals': {},
227
'rules': {}
238
};
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
module.exports = {
22
'extends': [
3-
// Need to qualify these here for ESLint to resolve them properly.
4-
'airbnb/base',
5-
'airbnb/rules/react.js'
6-
]
3+
'eslint-config-airbnb/base',
4+
'eslint-config-airbnb/rules/react',
5+
],
6+
rules: {}
77
};
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
module.exports = {
2+
'extends': [
3+
'eslint-config-airbnb/rules/best-practices',
4+
'eslint-config-airbnb/rules/errors',
5+
'eslint-config-airbnb/rules/legacy',
6+
'eslint-config-airbnb/rules/node',
7+
'eslint-config-airbnb/rules/strict',
8+
'eslint-config-airbnb/rules/style',
9+
'eslint-config-airbnb/rules/variables'
10+
],
11+
'env': {
12+
'browser': true,
13+
'node': true,
14+
'amd': false,
15+
'mocha': false,
16+
'jasmine': false
17+
},
18+
'ecmaFeatures': {},
19+
'globals': {},
20+
'rules': {}
21+
};

0 commit comments

Comments
 (0)