Permalink
Browse files

chore: Update dependencies to enable Greenkeeper ๐ŸŒด (#294)

* chore(package): update dependencies

https://greenkeeper.io/

* chore: Use peer dependency version

* style: Configure eslint-plugin-import and react plugin properly

* style: Use path.resolve instead of concatenation in webpack.config

* style: Fixed ESLint error in test.js

* style: Added eslint ignore

* chore: Fixed .eslintrc spec paths

* chore: Updated eslint channel on codeclimate

* chore: Updated React depenedencies
  • Loading branch information...
1 parent b2a4486 commit 95d632363b329b3fede19844b159f7d8ab46612a @greenkeeper greenkeeper committed with Dec 11, 2016
Showing with 97 additions and 43 deletions.
  1. +1 โˆ’1 .codeclimate.yml
  2. +80 โˆ’27 .eslintrc
  3. +10 โˆ’10 package.json
  4. +1 โˆ’1 src/index.js
  5. +4 โˆ’3 src/index.spec.js
  6. +1 โˆ’1 webpack.config.js
View
@@ -6,7 +6,7 @@ engines:
- javascript
eslint:
enabled: true
- channel: "eslint-2"
+ channel: "eslint-3"
ratings:
paths:
- "**.js"
View
@@ -1,29 +1,82 @@
{
- "parser": "babel-eslint",
- "extends": ["airbnb"],
- "env": {
- "browser": true, // We write for browser
- "node": true, // using CommonJS imports
- "jest": true, // We use Mocha for tests
- "es6": true // And we use ES6 features
- },
- "ecmaFeatures": {
- "jsx": true
- },
- "rules": {
- "indent": [2, 2, {
- "SwitchCase": 1
- }],
- "comma-dangle": [2, "never"],
- "jsx-quotes": [2, "prefer-double"],
- "padded-blocks": [0, "never"],
- "no-console": 2,
- "no-debugger": 2,
- "guard-for-in": 0,
- "object-curly-spacing": [2, "always"],
- "max-len": [2, {
- "code": 120,
- "ignoreComments": true
- }]
- }
+ "parser": "babel-eslint",
+ "extends": [
+ "airbnb"
+ ],
+ "env": {
+ // We write for browser
+ "browser": true,
+ // using CommonJS imports
+ "node": true,
+ // We use Jest for tests
+ "jest": true,
+ // And we use ES6 features
+ "es6": true
+ },
+ "ecmaFeatures": {
+ "jsx": true
+ },
+ "rules": {
+ "indent": [
+ 2,
+ 2,
+ {
+ "SwitchCase": 1
+ }
+ ],
+ "comma-dangle": [
+ 2,
+ "never"
+ ],
+ "jsx-quotes": [
+ 2,
+ "prefer-double"
+ ],
+ "padded-blocks": [
+ 0,
+ "never"
+ ],
+ "no-console": 2,
+ "no-debugger": 2,
+ "no-plusplus": 0,
+ "guard-for-in": 0,
+ "object-curly-spacing": [
+ 2,
+ "always"
+ ],
+ "max-len": [
+ 2,
+ {
+ "code": 120,
+ "ignoreComments": true
+ }
+ ],
+
+ // React
+ "react/jsx-filename-extension": 0,
+ "react/forbid-prop-types": [
+ 2,
+ {
+ "forbid": [
+ "any",
+ "array"
+ ]
+ }
+ ],
+
+ // a11y
+ "jsx-a11y/no-static-element-interactions": 0,
+
+ // Import
+ "import/no-extraneous-dependencies": [
+ 2,
+ {
+ "devDependencies": [
+ "**/webpack*.js",
+ "**/*.spec.js",
+ "./mocha-environment.js"
+ ]
+ }
+ ]
+ }
}
View
@@ -57,7 +57,7 @@
"devDependencies": {
"babel-cli": "^6.9.0",
"babel-core": "^6.9.1",
- "babel-eslint": "^6.0.4",
+ "babel-eslint": "^7.1.1",
"babel-jest": "^17.0.2",
"babel-loader": "^6.2.2",
"babel-plugin-add-module-exports": "^0.2.1",
@@ -66,21 +66,21 @@
"babel-preset-stage-0": "^6.3.13",
"babel-register": "^6.9.0",
"cz-conventional-changelog": "^1.2.0",
- "eslint": "^2.11.0",
- "eslint-config-airbnb": "^9.0.1",
- "eslint-plugin-import": "^1.8.1",
- "eslint-plugin-jsx-a11y": "^1.2.2",
- "eslint-plugin-react": "^5.1.1",
+ "eslint": "^3.11.0",
+ "eslint-config-airbnb": "^13.0.0",
+ "eslint-plugin-import": "^2.2.0",
+ "eslint-plugin-jsx-a11y": "^2.2.3",
+ "eslint-plugin-react": "^6.7.1",
"jest": "^17.0.3",
"lint-staged": "^3.2.1",
"npm-check": "^5.2.1",
"pre-commit": "^1.1.3",
- "react": "^15.2.0",
- "react-addons-test-utils": "^15.0.1",
- "react-dom": "^15.2.0",
+ "react": "^15.4.1",
+ "react-addons-test-utils": "^15.4.1",
+ "react-dom": "^15.4.1",
"react-testutils-additions": "^15.0.0",
"rimraf": "^2.5.2",
- "semantic-release": "^4.3.5",
+ "semantic-release": "^6.3.2",
"sinon": "^1.17.4",
"webpack": "^1.13.1"
},
View
@@ -64,7 +64,7 @@ class Dropzone extends React.Component {
}
}
- onDragOver(e) {
+ onDragOver(e) { // eslint-disable-line class-methods-use-this
e.preventDefault();
e.stopPropagation();
try {
View
@@ -3,7 +3,8 @@
import React from 'react';
import { spy } from 'sinon';
import TestUtils from 'react-testutils-additions';
-const Dropzone = require(process.env.NODE_ENV === 'production' ? '../dist/index' : './index');
+
+const Dropzone = require(process.env.NODE_ENV === 'production' ? '../dist/index' : './index'); // eslint-disable-line import/no-dynamic-require
describe('Dropzone', () => {
@@ -127,9 +128,9 @@ describe('Dropzone', () => {
it('renders dynamic props on the root element', () => {
const component = TestUtils.renderIntoDocument(
- <Dropzone hidden aria-hidden="hidden" title="Dropzone" />
+ <Dropzone hidden aria-hidden title="Dropzone" />
);
- expect(TestUtils.find(component, '[hidden][aria-hidden="hidden"][title="Dropzone"]'))
+ expect(TestUtils.find(component, '[hidden][aria-hidden][title="Dropzone"]'))
.toHaveLength(1);
});
View
@@ -7,7 +7,7 @@ module.exports = {
entry: './src/index.js',
devtool: 'source-map',
output: {
- path: __dirname + '/dist/',
+ path: path.resolve(__dirname, 'dist'),
filename: 'index.js',
libraryTarget: 'umd',
library: 'Dropzone'

0 comments on commit 95d6323

Please sign in to comment.