chore: Add linting

Author: davesnx

.eslintignore

@@ -0,0 +1,3 @@
+**/node_modules/**
+**/lib/**
+test/fixtures/**

.eslintrc.js

@@ -0,0 +1,3 @@
+module.exports = {
+  extends: 'davesnx-rules'
+}

package.json

@@ -14,11 +14,13 @@
     "build": "babel src -d lib",
     "test": "mocha --compilers js:babel-register",
     "test:watch": "yarn test -- --watch",
+    "lint": "eslint .",
     "prepublishOnly": "yarn clean && yarn build",
     "semantic-release": "semantic-release",
     "travis-deploy-once": "travis-deploy-once"
   },
   "dependencies": {
+    "eslint-config-davesnx-rules": "^1.0.0",
     "lodash.camelcase": "^4.3.0",
     "lodash.isstring": "^4.0.1",
     "lodash.kebabcase": "^4.1.1",
@@ -55,6 +57,7 @@
   ],
   "husky": {
     "hooks": {
+      "pre-commit": "npm run lint",
       "pre-push": "npm run test"
     }
   }

src/index.js

@@ -1,26 +1,26 @@
 import checkValidOptions from './options'
 import TaggedTemplateExpression from './styled-components'
 
-function isReactFragment (openingElement) {
+function isReactFragment(openingElement) {
   return (
     openingElement.node.name.name === 'Fragment' ||
     (openingElement.node.name.type === 'JSXMemberExpression' &&
       openingElement.node.name.object.name === 'React' &&
       openingElement.node.name.property.name === 'Fragment')
-  );
+  )
 }
 
-function functionBodyPushAttributes (t, path, options, componentName) {
+function functionBodyPushAttributes(t, path, options, componentName) {
   let openingElement = null
   const functionBody = path.get('body').get('body')
   if (functionBody.parent && functionBody.parent.type === 'JSXElement') {
-    const jsxElement = functionBody.find((c) => {
+    const jsxElement = functionBody.find(c => {
       return c.type === 'JSXElement'
     })
     if (!jsxElement) return
     openingElement = jsxElement.get('openingElement')
   } else {
-    const returnStatement = functionBody.find((c) => {
+    const returnStatement = functionBody.find(c => {
       return c.type === 'ReturnStatement'
     })
     if (!returnStatement) return
@@ -42,30 +42,30 @@ function functionBodyPushAttributes (t, path, options, componentName) {
   )
 }
 
-export default function ({types: t}) {
+export default function({ types: t }) {
   return {
     visitor: {
       TaggedTemplateExpression,
-      FunctionDeclaration (path, state) {
-        if (!path.node.id || !path.node.id.name) return;
+      FunctionDeclaration(path, state) {
+        if (!path.node.id || !path.node.id.name) return
 
         const options = checkValidOptions(state)
         const componentName = path.node.id.name
 
         functionBodyPushAttributes(t, path, options, componentName)
       },
-      ArrowFunctionExpression (path, state) {
+      ArrowFunctionExpression(path, state) {
         const options = checkValidOptions(state)
         if (!path.parent.id || !path.parent.id.name) return
         const componentName = path.parent.id.name
 
         functionBodyPushAttributes(t, path, options, componentName)
       },
-      ClassDeclaration (path, state) {
-        let name = path.get('id')
-        let properties = path.get('body').get('body')
+      ClassDeclaration(path, state) {
+        const name = path.get('id')
+        const properties = path.get('body').get('body')
 
-        let render = properties.find(prop => {
+        const render = properties.find(prop => {
           return (
             prop.isClassMethod() &&
             prop.get('key').isIdentifier({ name: 'render' })
@@ -79,7 +79,7 @@ export default function ({types: t}) {
         const options = checkValidOptions(state)
 
         render.traverse({
-          ReturnStatement (returnStatement) {
+          ReturnStatement(returnStatement) {
             const arg = returnStatement.get('argument')
             if (!arg.isJSXElement()) return
 

src/styled-components.js

@@ -31,13 +31,14 @@ export default function TaggedTemplateExpression(path, state) {
     }
   } catch (e) {}
 
-  return
-
   // hoisted helpers in closure
   function insertBefore(node, id) {
     return t.callExpression(t.memberExpression(node, t.identifier('attrs')), [
       t.objectExpression([
-        t.objectProperty(t.StringLiteral(options.attribute), t.StringLiteral(options.format(id)))
+        t.objectProperty(
+          t.StringLiteral(options.attribute),
+          t.StringLiteral(options.format(id))
+        )
       ])
     ])
   }
@@ -51,7 +52,7 @@ export default function TaggedTemplateExpression(path, state) {
       if (!t.isIdentifier(node)) return false
       const binding = scope.getBinding(node.name)
       if (!binding || binding.kind !== 'module') return false
-      return binding.path.parent.source.value == 'styled-components'
+      return binding.path.parent.source.value === 'styled-components'
     }
   }
   function getIdFrom(parentPath) {