Update linting dependencies, fix errors

zpao · zpao · commit 52c4cbad426c · 2015-09-22T14:29:04.000-07:00
diff --git a/.eslintrc b/.eslintrc
@@ -1,3 +1,7 @@
 ---
 "extends":
-  - "defaults/configurations/walmart/es6-react"
+  - "eslint-config-defaults/configurations/walmart/es6-react"
+
+"rules":
+  "indent": [2, 2, {"SwitchCase": 1}]
+  "max-len": 0
diff --git a/package.json b/package.json
@@ -30,12 +30,13 @@
     "autoprefixer-core": "^5.2.1",
     "babel": "^5.6.14",
     "babel-core": "^5.6.15",
-    "babel-eslint": "^3.1.23",
+    "babel-eslint": "^4.1.3",
     "babel-loader": "^5.2.2",
     "css-loader": "^0.15.1",
-    "eslint": "^0.24.0",
-    "eslint-config-defaults": "^3.0.3",
-    "eslint-plugin-react": "^2.6.4",
+    "eslint": "^1.5.1",
+    "eslint-config-defaults": "^6.0.0",
+    "eslint-plugin-filenames": "^0.1.2",
+    "eslint-plugin-react": "^3.4.2",
     "file-loader": "^0.8.4",
     "json-loader": "^0.5.2",
     "node-libs-browser": "^0.5.2",
diff --git a/src/appear.jsx b/src/appear.jsx
@@ -6,7 +6,8 @@ import assign from "object-assign";
 const Appear = React.createClass({
   mixins: [tweenState.Mixin],
   propTypes: {
-    children: React.PropTypes.node
+    children: React.PropTypes.node,
+    style: React.PropTypes.object
   },
   contextTypes: {
     flux: React.PropTypes.object,
diff --git a/src/base.jsx b/src/base.jsx
@@ -1,4 +1,4 @@
-/*eslint max-statements:0,complexity:0*/
+/*eslint max-statements:0,complexity:0,no-invalid-this:0*/
 import React from "react/addons";
 
 const getStyles = function getStyles() {
diff --git a/src/block-quote.jsx b/src/block-quote.jsx
@@ -14,7 +14,8 @@ class BlockQuote extends Base {
 }
 
 BlockQuote.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 BlockQuote.contextTypes = {
diff --git a/src/code-pane.jsx b/src/code-pane.jsx
@@ -29,7 +29,8 @@ CodePane.contextTypes = {
 
 CodePane.propTypes = {
   lang: React.PropTypes.string,
-  source: React.PropTypes.string
+  source: React.PropTypes.string,
+  style: React.PropTypes.object
 };
 
 CodePane.defaultProps = {
diff --git a/src/code.jsx b/src/code.jsx
@@ -14,7 +14,8 @@ class Code extends Base {
 }
 
 Code.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 Code.contextTypes = {
diff --git a/src/deck.jsx b/src/deck.jsx
@@ -128,7 +128,7 @@ class Deck extends React.Component {
   }
   _getHash(slide) {
     let hash = slide;
-    if ('id' in this.props.children[slide].props) {
+    if ("id" in this.props.children[slide].props) {
       hash = this.props.children[slide].props.id;
     }
     return hash;
diff --git a/src/fill.jsx b/src/fill.jsx
@@ -16,7 +16,8 @@ class Fill extends React.Component {
 }
 
 Fill.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 export default Fill;
diff --git a/src/fit.jsx b/src/fit.jsx
@@ -16,7 +16,8 @@ class Fit extends React.Component {
 }
 
 Fit.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 export default Fit;
diff --git a/src/heading.jsx b/src/heading.jsx
@@ -44,18 +44,18 @@ class Heading extends Base {
     const Tag = "H" + this.props.size;
     const styles = {
       container: {
-        display: 'block',
-        width: '100%',
+        display: "block",
+        width: "100%",
         height: this.state.height
       },
       text: {
         fontSize: 16,
-        display: 'block',
+        display: "block",
         margin: "0",
         padding: "0",
         lineHeight: this.props.lineHeight,
         transform: "scale(" + this.state.scale + ")",
-        transformOrigin: 'center top'
+        transformOrigin: "center top"
       }
     };
     return this.props.fit
@@ -71,8 +71,8 @@ class Heading extends Base {
         </span>
       </div>
     : React.createElement(Tag, {
-        style: [this.context.styles.components.heading["h" + this.props.size], this.getStyles(), this.props.style]
-      }, this.props.children);
+      style: [this.context.styles.components.heading["h" + this.props.size], this.getStyles(), this.props.style]
+    }, this.props.children);
   }
 }
 
diff --git a/src/image.jsx b/src/image.jsx
@@ -32,7 +32,8 @@ Image.propTypes = {
     React.PropTypes.number
   ]),
   display: React.PropTypes.string,
-  src: React.PropTypes.string
+  src: React.PropTypes.string,
+  style: React.PropTypes.object
 };
 
 Image.contextTypes = {
diff --git a/src/layout.jsx b/src/layout.jsx
@@ -16,7 +16,8 @@ class Layout extends React.Component {
 }
 
 Layout.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 export default Layout;
diff --git a/src/link.jsx b/src/link.jsx
@@ -15,7 +15,8 @@ class Link extends Base {
 
 Link.propTypes = {
   children: React.PropTypes.node,
-  href: React.PropTypes.string
+  href: React.PropTypes.string,
+  style: React.PropTypes.object
 };
 
 Link.contextTypes = {
diff --git a/src/list-item.jsx b/src/list-item.jsx
@@ -14,7 +14,8 @@ class ListItem extends Base {
 }
 
 ListItem.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 ListItem.contextTypes = {
diff --git a/src/list.jsx b/src/list.jsx
@@ -14,7 +14,8 @@ class List extends Base {
 }
 
 List.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 List.contextTypes = {
diff --git a/src/overview.jsx b/src/overview.jsx
@@ -1,5 +1,3 @@
-/*global setInterval*/
-
 import React from "react/addons";
 import cloneWithProps from "react/lib/cloneWithProps";
 import Base from "./base";
diff --git a/src/presenter.jsx b/src/presenter.jsx
@@ -29,7 +29,7 @@ class Presenter extends Base {
   _renderMainSlide() {
     const child = this.props.slides[this.props.slide];
     const presenterStyle = {
-      position: "relative",
+      position: "relative"
     };
     return cloneWithProps(child, {
       key: this.props.slide,
@@ -42,7 +42,7 @@ class Presenter extends Base {
     });
   }
   componentDidMount() {
-    this.time = setInterval(()=> {
+    this.time = setInterval(() => {
       this.setState({
         time: startTime(new Date())
       });
@@ -61,7 +61,7 @@ class Presenter extends Base {
       left: "50%",
       transform: "translate(-50%, -50%)",
       margin: 0,
-      color: 'white'
+      color: "white"
     };
     const child = this.props.slides[parseInt(this.props.slide) + 1];
     return child ? cloneWithProps(child, {
@@ -183,6 +183,7 @@ class Presenter extends Base {
 
 Presenter.propTypes = {
   lastSlide: React.PropTypes.number,
+  hash: React.PropTypes.number,
   slides: React.PropTypes.array,
   slide: React.PropTypes.number
 };
diff --git a/src/quote.jsx b/src/quote.jsx
@@ -14,7 +14,8 @@ class Quote extends Base {
 }
 
 Quote.propTypes = {
-  children: React.PropTypes.node
+  children: React.PropTypes.node,
+  style: React.PropTypes.object
 };
 
 Quote.contextTypes = {
diff --git a/src/s.jsx b/src/s.jsx
@@ -28,6 +28,7 @@ class S extends React.Component {
 
 S.propTypes = {
   children: React.PropTypes.node,
+  style: React.PropTypes.object,
   type: React.PropTypes.array
 };
 
diff --git a/src/slide.jsx b/src/slide.jsx
@@ -18,6 +18,7 @@ const Slide = React.createClass({
   },
   propTypes: {
     align: React.PropTypes.string,
+    hash: React.PropTypes.number,
     presenterStyle: React.PropTypes.object,
     children: React.PropTypes.node,
     notes: React.PropTypes.string,
@@ -76,12 +77,12 @@ const Slide = React.createClass({
     } : {};
     const overViewStyles = {
       inner: {
-        flexDirection: 'column'
+        flexDirection: "column"
       },
       content: {
         width: "100%"
       }
-    }
+    };
     const styles = {
       outer: {
         position: this.context.export ? "relative" : "absolute",
diff --git a/src/text.jsx b/src/text.jsx
@@ -43,18 +43,18 @@ class Text extends Base {
   render() {
     const styles = {
       container: {
-        display: 'block',
-        width: '100%',
+        display: "block",
+        width: "100%",
         height: this.state.height
       },
       text: {
         fontSize: 16,
-        display: 'block',
+        display: "block",
         margin: "0",
         padding: "0",
         lineHeight: this.props.lineHeight,
         transform: "scale(" + this.state.scale + ")",
-        transformOrigin: 'center top'
+        transformOrigin: "center top"
       }
     };
     return this.props.fit
diff --git a/src/transitions.jsx b/src/transitions.jsx
@@ -52,7 +52,7 @@ export default {
 
     this.setState({
       z: 1
-    }, ()=> {
+    }, () => {
 
       if (this.props.transition.indexOf("fade") !== -1) {
         this.tweenState("opacity", {
diff --git a/src/utils/context.jsx b/src/utils/context.jsx
@@ -35,7 +35,7 @@ const context = function context(Component, params) {
         overview: location.query && "overview" in location.query,
         export: location.query && "export" in location.query,
         print: location.query && "print" in location.query,
-        slide: slide
+        slide
       };
     },
 

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-/eslint max-statements:0,complexity:0/`
	`1`	`+/eslint max-statements:0,complexity:0,no-invalid-this:0/`
`2`	`2`	`import React from "react/addons";`
`3`	`3`
`4`	`4`	`const getStyles = function getStyles() {`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,8 @@ class BlockQuote extends Base {`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`BlockQuote.propTypes = {`
`17`		`- children: React.PropTypes.node`
	`17`	`+ children: React.PropTypes.node,`
	`18`	`+ style: React.PropTypes.object`
`18`	`19`	`};`
`19`	`20`
`20`	`21`	`BlockQuote.contextTypes = {`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,8 @@ class Code extends Base {`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`Code.propTypes = {`
`17`		`- children: React.PropTypes.node`
	`17`	`+ children: React.PropTypes.node,`
	`18`	`+ style: React.PropTypes.object`
`18`	`19`	`};`
`19`	`20`
`20`	`21`	`Code.contextTypes = {`
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ class Deck extends React.Component {`
`128`	`128`	`}`
`129`	`129`	`_getHash(slide) {`
`130`	`130`	`let hash = slide;`
`131`		`- if ('id' in this.props.children[slide].props) {`
	`131`	`+ if ("id" in this.props.children[slide].props) {`
`132`	`132`	`hash = this.props.children[slide].props.id;`
`133`	`133`	`}`
`134`	`134`	`return hash;`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,8 @@ class Fill extends React.Component {`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`Fill.propTypes = {`
`19`		`- children: React.PropTypes.node`
	`19`	`+ children: React.PropTypes.node,`
	`20`	`+ style: React.PropTypes.object`
`20`	`21`	`};`
`21`	`22`
`22`	`23`	`export default Fill;`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,8 @@ class Fit extends React.Component {`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`Fit.propTypes = {`
`19`		`- children: React.PropTypes.node`
	`19`	`+ children: React.PropTypes.node,`
	`20`	`+ style: React.PropTypes.object`
`20`	`21`	`};`
`21`	`22`
`22`	`23`	`export default Fit;`