Add ESLint support

.eslintignore

@@ -0,0 +1,2 @@
+**/node_modules/*
+**/coverage/*

.eslintrc.json

@@ -0,0 +1,12 @@
+{
+  "extends": [
+    "airbnb",
+    "prettier",
+    "prettier/react"
+  ],
+  "rules": {
+    "react/jsx-filename-extension": [1, { "extensions": [".js"] }],
+    "import/extensions": "off"
+  },
+  "root": true
+}

.storybook.native/index.android.js

@@ -1,16 +1,16 @@
-import { AppRegistry, NativeModules } from 'react-native';
-import { getStorybookUI, configure } from '@storybook/react-native';
-import { loadStories } from './story-loader';
-import './addons';
-import url from 'url';
+import url from "url";
+import { AppRegistry, NativeModules } from "react-native";
+import { getStorybookUI, configure } from "@storybook/react-native";
+import { loadStories } from "./story-loader";
+import "./addons";
 
 configure(loadStories, module);
 
 const { hostname } = url.parse(NativeModules.SourceCode.scriptURL);
 
 const StorybookUI = getStorybookUI({
   port: 7007,
-  host: hostname,
+  host: hostname
 });
 
-AppRegistry.registerComponent('storybooknative', () => StorybookUI);
+AppRegistry.registerComponent("storybooknative", () => StorybookUI);

.storybook.native/index.ios.js

@@ -1,16 +1,16 @@
-import { AppRegistry, NativeModules } from 'react-native';
-import { getStorybookUI, configure } from '@storybook/react-native';
-import { loadStories } from './story-loader';
-import './addons';
-import url from 'url';
+import url from "url";
+import { AppRegistry, NativeModules } from "react-native";
+import { getStorybookUI, configure } from "@storybook/react-native";
+import { loadStories } from "./story-loader";
+import "./addons";
 
 configure(loadStories, module);
 
 const { hostname } = url.parse(NativeModules.SourceCode.scriptURL);
 
 const StorybookUI = getStorybookUI({
   port: 7007,
-  host: hostname,
+  host: hostname
 });
 
-AppRegistry.registerComponent('storybooknative', () => StorybookUI);
+AppRegistry.registerComponent("storybooknative", () => StorybookUI);

lib/cli/cli.js

@@ -1,5 +1,3 @@
-"use strict";
-
 const fs = require("fs");
 const path = require("path");
 
@@ -11,30 +9,17 @@ const fileTemplates = require("./templates");
 
 const repositoryRoot = path.join(__dirname, "../../");
 
-module.exports = () => {
-  subcommander.command("add component", {
-    desc: "create a new component",
-    callback: addComponent
+function writeStubFiles(variables) {
+  const packageRoot = path.join(
+    repositoryRoot,
+    "packages",
+    variables.packageName
+  );
+  mkdirp.sync(packageRoot);
+  fileTemplates(variables).forEach(([template, fileName]) => {
+    fs.writeFileSync(path.join(packageRoot, fileName), template(variables));
   });
-
-  const multiSubCommand = ["add component"].filter(command => {
-    const subcommands = command.split(" ").length;
-    return process.argv.slice(2, 2 + subcommands).join(" ") === command;
-  })[0];
-
-  if (multiSubCommand) {
-    const matchLength = multiSubCommand.split(" ").length;
-    process.argv = process.argv
-      .slice(0, 2)
-      .concat(process.argv.slice(2, 2 + matchLength).join(" "))
-      .concat(process.argv.slice(2 + matchLength));
-  } else {
-    subcommander.usage();
-    process.exit(1);
-  }
-
-  subcommander.parse();
-};
+}
 
 function addComponent() {
   const component = process.argv[3];
@@ -44,7 +29,7 @@ function addComponent() {
     '\n\nUsage:\n$ times-components add component ComponentName "Component Description"';
   const argumentOrUsage = (argument, message) => {
     if (!argument) {
-      console.error(message + usage);
+      console.error(message + usage); // eslint-disable-line no-console
       process.exit(1);
     }
   };
@@ -68,14 +53,27 @@ function addComponent() {
   });
 }
 
-function writeStubFiles(variables) {
-  const packageRoot = path.join(
-    repositoryRoot,
-    "packages",
-    variables.packageName
-  );
-  mkdirp.sync(packageRoot);
-  fileTemplates(variables).forEach(([template, fileName]) => {
-    fs.writeFileSync(path.join(packageRoot, fileName), template(variables));
+module.exports = () => {
+  subcommander.command("add component", {
+    desc: "create a new component",
+    callback: addComponent
   });
-}
+
+  const multiSubCommand = ["add component"].filter(command => {
+    const subcommands = command.split(" ").length;
+    return process.argv.slice(2, 2 + subcommands).join(" ") === command;
+  })[0];
+
+  if (multiSubCommand) {
+    const matchLength = multiSubCommand.split(" ").length;
+    process.argv = process.argv
+      .slice(0, 2)
+      .concat(process.argv.slice(2, 2 + matchLength).join(" "))
+      .concat(process.argv.slice(2 + matchLength));
+  } else {
+    subcommander.usage();
+    process.exit(1);
+  }
+
+  subcommander.parse();
+};

lib/cli/cli.test.js

@@ -1,3 +1,5 @@
+/* eslint-env jest */
+
 import path from "path";
 import fs from "fs";
 

lib/cli/package-json.js

@@ -1,10 +1,8 @@
-"use strict";
-
 module.exports = variables => ({
-  name: "@timescomponents/" + variables.packageName,
+  name: `@timescomponents/${variables.packageName}`,
   version: variables.version || "0.0.1",
   description: variables.packageDescription,
-  main: variables.packageName + ".js",
+  main: `${variables.packageName}.js`,
   scripts: {
     flow: "node_modules/flow-bin/cli.js",
     "test:watch": `jest ${variables.packageName}.test.js --bail --verbose --watch`,
@@ -48,5 +46,8 @@ module.exports = variables => ({
     react: "16.0.0-alpha.6",
     "react-dom": "15.5.4",
     "react-native-web": "0.0.97"
+  },
+  peerDependencies: {
+    "@storybook/react-native": "3.1.6"
   }
 });

lib/cli/templates.js

@@ -1,5 +1,3 @@
-"use strict";
-
 const handlebars = require("handlebars");
 
 const packageJson = require("./package-json");
@@ -24,7 +22,9 @@ storiesOf("{{{component}}}", module).add("{{{component}}}", () =>
 );
 `);
 
-const testFile = handlebars.compile(`import "react-native";
+const testFile = handlebars.compile(`/* eslint-env jest */
+
+import "react-native";
 import React from "react";
 import {{{component}}} from "./{{{packageName}}}";
 import renderer from "react-test-renderer";
@@ -71,9 +71,9 @@ const packageJsonFile = variables =>
   JSON.stringify(packageJson(variables), null, 2);
 
 module.exports = variables => [
-  [indexFile, variables.packageName + ".js"],
-  [testFile, variables.packageName + ".test.js"],
-  [storyFile, variables.packageName + ".stories.js"],
+  [indexFile, `${variables.packageName}.js`],
+  [testFile, `${variables.packageName}.test.js`],
+  [storyFile, `${variables.packageName}.stories.js`],
   [licenseFile, "LICENSE"],
   [packageJsonFile, "package.json"]
 ];

package.json

@@ -32,6 +32,7 @@
   ],
   "lint-staged": {
     "**/*.js": [
+      "eslint . --fix",
       "prettier --write",
       "git add"
     ]
@@ -64,26 +65,32 @@
     "@storybook/addon-actions": "3.0.0",
     "@storybook/react": "3.0.0",
     "@storybook/react-native": "3.0.0",
-    "coveralls": "^2.13.1",
-    "jest": "^20.0.4",
-    "lcov-result-merger": "^1.2.0",
+    "coveralls": "2.13.1",
+    "eslint": "3.19.0",
+    "eslint-config-airbnb": "15.0.1",
+    "eslint-config-prettier": "2.3.0",
+    "eslint-plugin-import": "2.6.1",
+    "eslint-plugin-jsx-a11y": "5.1.1",
+    "eslint-plugin-react": "7.1.0",
+    "jest": "20.0.4",
+    "lcov-result-merger": "1.2.0",
     "lerna": "2.0.0-rc.5",
-    "lint-staged": "^4.0.0",
-    "pre-commit": "^1.2.2",
+    "lint-staged": "4.0.0",
+    "pre-commit": "1.2.2",
     "prettier": "1.4.2",
     "react": "16.0.0-alpha.6",
     "react-dom": "15.5.4",
-    "react-native": "0.44.2",
+    "react-native": "0.44.1",
     "react-native-storybook-loader": "1.3.1",
     "react-native-web": "0.0.97",
-    "rimraf": "^2.6.1",
+    "rimraf": "2.6.1",
     "url": "0.11.0"
   },
   "dependencies": {
     "dashify": "0.2.2",
     "global": "4.3.2",
     "handlebars": "4.0.10",
-    "mkdirp": "^0.5.1",
+    "mkdirp": "0.5.1",
     "sort-pkg": "1.1.0",
     "subcommander": "1.0.0"
   }

packages/author/author.js

@@ -1,18 +1,6 @@
-import React, { Component } from "react";
+import React from "react";
 import { StyleSheet, Text, View } from "react-native";
 
-export default class Author extends Component {
-  render() {
-    return (
-      <View style={styles.container}>
-        <Text>
-          Author Bios!
-        </Text>
-      </View>
-    );
-  }
-}
-
 const styles = StyleSheet.create({
   container: {
     flex: 1,
@@ -21,3 +9,12 @@ const styles = StyleSheet.create({
     backgroundColor: "#F5EFEB"
   }
 });
+
+const Author = () =>
+  <View style={styles.container}>
+    <Text>
+      Author Bios!
+    </Text>
+  </View>;
+
+export default Author;

packages/author/author.test.js

@@ -1,8 +1,11 @@
+/* eslint-env jest */
+
 import "react-native";
 import React from "react";
-import Author from "./author";
 import renderer from "react-test-renderer";
 
+import Author from "./author";
+
 it("renders correctly", () => {
   const tree = renderer.create(<Author />).toJSON();
 

packages/author/package.json

@@ -37,13 +37,16 @@
     "babel-preset-react-native": "1.9.2",
     "flow-bin": "0.42.0",
     "jest": "20.0.4",
-    "react-native": "0.44.2",
     "react-test-renderer": "15.5.4",
     "webpack": "2.6.1"
   },
   "dependencies": {
     "react": "16.0.0-alpha.6",
     "react-dom": "15.5.4",
+    "react-native": "0.44.1",
     "react-native-web": "0.0.96"
+  },
+  "peerDependencies": {
+    "@storybook/react-native": "3.1.6"
   }
 }