Fix linting issues related to postcss-cssnext merge

MoOx · MoOx · commit b9076ee7ba00 · 2016-01-02T22:59:21.000+01:00
diff --git a/.eslintrc b/.eslintrc
@@ -35,6 +35,7 @@ rules:
   indent: [2, 2] # 2 spaces indentation
   max-len: [2, 80, 4]
   quotes: [2, "double"]
+  jsx-quotes: [2, "prefer-double"]
   semi: [2, "never"]
   no-multiple-empty-lines: [2, {"max": 1}]
 
@@ -65,7 +66,6 @@ rules:
   # eslint-plugin-react rules
   react/jsx-boolean-value: 2
   react/jsx-no-undef: 2
-  react/jsx-quotes: 2
   #react/jsx-sort-prop-types: 2
   #react/jsx-sort-props: 2
   react/jsx-uses-react: 2
diff --git a/docs/scripts/build.js b/docs/scripts/build.js
@@ -1,6 +1,6 @@
 import path from "path"
 
-import {sync as rm} from "rimraf"
+import { sync as rm } from "rimraf"
 import color from "chalk"
 
 import Metalsmith from "metalsmith"
@@ -30,7 +30,7 @@ const log = nanoLogger("./build")
 
 JSON.stringify(buildConfig, null, 2).split("\n").forEach(l => log(l))
 
-const mdToHtmlReplacement = [/\.md$/, ".html"]
+const mdToHtmlReplacement = [ /\.md$/, ".html" ]
 
 // We clean ./dist by hand mainly for prod, in order to be able to build
 // assets with webpack before metalsmith build.
@@ -63,7 +63,7 @@ smith
 .use(
   url([
     mdToHtmlReplacement,
-    [/index\.html?$/, ""],
+    [ /index\.html?$/, "" ],
   ])
 )
 // wrap .html into react `template:`
@@ -87,9 +87,9 @@ smith
   rename([
     mdToHtmlReplacement,
     // no .html at the end of urls
-    [/\.html$/, "/index.html"],
+    [ /\.html$/, "/index.html" ],
     // ensure we only have index.html, no index/index
-    [/index\/index\.html$/, "index.html"],
+    [ /index\/index\.html$/, "index.html" ],
   ])
 )
 
diff --git a/docs/scripts/webpack.js b/docs/scripts/webpack.js
@@ -9,13 +9,19 @@ export default (webpackConfig, log, cb) => {
 
     if (stats.hasErrors()) {
       stats.compilation.errors.forEach(
-        item => log(...[color.red("Error:"), ...item.message.split("\n")])
+        item => log(...[
+          color.red("Error:"),
+          ...item.message.split("\n"),
+        ])
       )
       throw new Error("webpack build failed with errors")
     }
     if (stats.hasWarnings()) {
       stats.compilation.warnings.forEach(
-        item => log(...[color.yellow("Warning:"), ...item.message.split("\n")])
+        item => log(...[
+          color.yellow("Warning:"),
+          ...item.message.split("\n"),
+        ])
       )
     }
 
diff --git a/docs/src/layouts/Default.js b/docs/src/layouts/Default.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 import cx from "classnames"
 
 import dashify from "../modules/dashify"
@@ -124,7 +124,7 @@ export default class Default extends Component {
                 "r-Grid-cell": true,
                 "js-markdownIt-TOCOriginalContainer": true,
               })}
-              dangerouslySetInnerHTML={{__html: file.contents}}
+              dangerouslySetInnerHTML={{ __html: file.contents }}
             />
             <div
               className={cx({
diff --git a/docs/src/layouts/Simple.js b/docs/src/layouts/Simple.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 
 import Html from "../modules/Html"
 import Head from "../modules/Head"
@@ -52,7 +52,7 @@ export default class Simple extends Component {
         >
           <div
             className={file.className || ""}
-            dangerouslySetInnerHTML={{__html: file.contents}}
+            dangerouslySetInnerHTML={{ __html: file.contents }}
           />
         </Body>
       </Html>
diff --git a/docs/src/modules/Analytics/index.js b/docs/src/modules/Analytics/index.js
@@ -1,4 +1,4 @@
-import React, {Component} from "react"
+import React, { Component } from "react"
 
 import isogram from "isogram"
 
diff --git a/docs/src/modules/Body/index.js b/docs/src/modules/Body/index.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 
 import Header from "../Header"
 import Footer from "../Footer"
diff --git a/docs/src/modules/Footer/index.js b/docs/src/modules/Footer/index.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 
 import SVGIcon from "../SVGIcon"
 import requireRaw from "../requireRaw"
diff --git a/docs/src/modules/Head/index.js b/docs/src/modules/Head/index.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 
 export default class Head extends Component {
 
diff --git a/docs/src/modules/Header/index.js b/docs/src/modules/Header/index.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 
 import SVGIcon from "../SVGIcon"
 import requireRaw from "../requireRaw"
diff --git a/docs/src/modules/Header/link.js b/docs/src/modules/Header/link.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 import cx from "classnames"
 
 export default class HeaderLink extends Component {
@@ -30,7 +30,7 @@ export default class HeaderLink extends Component {
         className={cx({
           ...(
             this.props.className
-              ? {[this.props.className]: true}
+              ? { [this.props.className]: true }
               : {}
           ),
           "cssnext-Header-nav-item": true,
diff --git a/docs/src/modules/Html/index.js b/docs/src/modules/Html/index.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 
 export default class Html extends Component {
 
diff --git a/docs/src/modules/SVGIcon/__tests__/index.js b/docs/src/modules/SVGIcon/__tests__/index.js
@@ -1,5 +1,5 @@
 import tape from "tape-catch"
-import React, {Component} from "react"
+import React, { Component } from "react"
 import SVGIcon from ".."
 
 tape("SVGIcon", (test) => {
@@ -70,7 +70,7 @@ tape("SVGIcon", (test) => {
 
   test.equal(
     React.renderToStaticMarkup(
-      <SVGIcon cleanupExceptions={["fill"]} svg={`<svg ${svgPiece}`} />
+      <SVGIcon cleanupExceptions={[ "fill" ]} svg={`<svg ${svgPiece}`} />
     ),
     `${SVGIconCleanedStart}><g fill="none"><path fill="#ab234f"></path></g>` +
       `</svg></span>`,
diff --git a/docs/src/modules/SVGIcon/index.js b/docs/src/modules/SVGIcon/index.js
@@ -1,4 +1,4 @@
-import React, {Component, PropTypes} from "react"
+import React, { Component, PropTypes } from "react"
 // import styled from "bloody-react-styled"
 import cx from "classnames"
 
@@ -28,14 +28,9 @@ const cleanups = {
 // @styled(styles)
 export default class SVGIcon extends Component {
 
-  static defaultProps = {
-    component: "span",
-    classSuffix: "-svg",
-    cleanup: [],
-    cleanupExceptions: [],
-  }
-
   static propTypes = {
+    className: PropTypes.string,
+    classSuffix: PropTypes.string,
     component: PropTypes.oneOfType([
       PropTypes.string,
       PropTypes.func,
@@ -46,10 +41,18 @@ export default class SVGIcon extends Component {
       PropTypes.bool,
       PropTypes.array,
     ]),
+    cleanupExceptions: PropTypes.array,
     width: PropTypes.string,
     height: PropTypes.string,
   }
 
+  static defaultProps = {
+    component: "span",
+    classSuffix: "-svg",
+    cleanup: [],
+    cleanupExceptions: [],
+  }
+
   static cleanupSvg(svg, cleanup = []) {
     return Object.keys(cleanups)
       .filter(key => cleanup.includes(key))
@@ -97,7 +100,7 @@ export default class SVGIcon extends Component {
       height = width
     }
 
-    const props = {...this.props}
+    const props = { ...this.props }
     // remove useless props for wrapper
     delete props.svg
     delete props.fill
diff --git a/docs/src/modules/playground/index.js b/docs/src/modules/playground/index.js
@@ -1,5 +1,5 @@
 import cssnext from "../../../../src/index"
-import {version as cssnextVersion} from "../../../../package"
+import { version as cssnextVersion } from "../../../../package"
 import messagesStyles from "../../../../src/option.messages.browser.styles"
 
 const bullet = "›"
diff --git a/docs/src/modules/requireRaw.js b/docs/src/modules/requireRaw.js
@@ -11,7 +11,7 @@ if (fs && fs.readFileSync) {
   requireRaw = (filename) => {
     return fs.readFileSync(
       path.join(__dirname, "..", "..", filename),
-      {encoding: "utf8"}
+      { encoding: "utf8" }
     )
   }
 }
diff --git a/webpack.config.js b/webpack.config.js
@@ -48,7 +48,7 @@ module.exports = {
 
   plugins: ([
     new webpack.DefinePlugin(buildConfig),
-    new ExtractTextPlugin("[name].css", {disable: !buildConfig.__PROD__}),
+    new ExtractTextPlugin("[name].css", { disable: !buildConfig.__PROD__ }),
   ].concat(
       buildConfig.__PROD__
       ? [

Original file line number	Diff line number	Diff line change
`@@ -9,13 +9,19 @@ export default (webpackConfig, log, cb) => {`
`9`	`9`
`10`	`10`	`if (stats.hasErrors()) {`
`11`	`11`	`stats.compilation.errors.forEach(`
`12`		`- item => log(...[color.red("Error:"), ...item.message.split("\n")])`
	`12`	`+ item => log(...[`
	`13`	`+ color.red("Error:"),`
	`14`	`+ ...item.message.split("\n"),`
	`15`	`+ ])`
`13`	`16`	`)`
`14`	`17`	`throw new Error("webpack build failed with errors")`
`15`	`18`	`}`
`16`	`19`	`if (stats.hasWarnings()) {`
`17`	`20`	`stats.compilation.warnings.forEach(`
`18`		`- item => log(...[color.yellow("Warning:"), ...item.message.split("\n")])`
	`21`	`+ item => log(...[`
	`22`	`+ color.yellow("Warning:"),`
	`23`	`+ ...item.message.split("\n"),`
	`24`	`+ ])`
`19`	`25`	`)`
`20`	`26`	`}`
`21`	`27`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import React, {Component} from "react"`
	`1`	`+import React, { Component } from "react"`
`2`	`2`
`3`	`3`	`import isogram from "isogram"`
`4`	`4`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import React, {Component, PropTypes} from "react"`
	`1`	`+import React, { Component, PropTypes } from "react"`
`2`	`2`
`3`	`3`	`import Header from "../Header"`
`4`	`4`	`import Footer from "../Footer"`