Skip to content

Commit

Permalink
[Tests] fix linting error caused by updated base config
Browse files Browse the repository at this point in the history
  • Loading branch information
ljharb committed Sep 2, 2017
1 parent 9d91990 commit 63d1ae1
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 6 deletions.
6 changes: 3 additions & 3 deletions packages/eslint-config-airbnb-base/rules/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ module.exports = {
body: 1
},
CallExpression: {
'arguments': 1
arguments: 1
},
ArrayExpression: 1,
ObjectExpression: 1,
Expand Down Expand Up @@ -344,8 +344,8 @@ module.exports = {
// enforce line breaks between braces
// https://eslint.org/docs/rules/object-curly-newline
'object-curly-newline': ['error', {
ObjectExpression: { minProperties: 3, multiline: true, consistent: true },
ObjectPattern: { minProperties: 3, multiline: true, consistent: true }
ObjectExpression: { minProperties: 4, multiline: true, consistent: true },
ObjectPattern: { minProperties: 4, multiline: true, consistent: true }
}],

// enforce "same line" or "multiple line" on object properties.
Expand Down
4 changes: 2 additions & 2 deletions packages/eslint-config-airbnb-base/test/test-base.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ fs.readdirSync(path.join(__dirname, '../rules')).forEach((name) => {
files[name] = require(`../rules/${name}`); // eslint-disable-line global-require
});

Object.keys(files).forEach((
Object.keys(files).forEach(( // eslint-disable-line function-paren-newline
name, // trailing function comma is to test parsing
) => {
) => { // eslint-disable-line function-paren-newline
const config = files[name];

test(`${name}: does not reference react`, (t) => {
Expand Down
6 changes: 5 additions & 1 deletion packages/eslint-config-airbnb/rules/react.js
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,11 @@ module.exports = {

// Prevent missing props validation in a React component definition
// https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/prop-types.md
'react/prop-types': ['error', { ignore: [], customValidators: [], skipUndeclared: false }],
'react/prop-types': ['error', {
ignore: [],
customValidators: [],
skipUndeclared: false
}],

// Prevent missing React when using JSX
// https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/react-in-jsx-scope.md
Expand Down

0 comments on commit 63d1ae1

Please sign in to comment.