[added] yarn run ci

Summary:
I checked out a fresh copy of the repo to ensure that it could pass CI.  Here are the changes I made:

- Moved the CI scripts from .travis.yml to package.json for consistency.  It's nice to be able to type `yarn run ci` to run the same suite as Travis.
- Ran `build` before `test`, to ensure any intra-repo dependencies are available in the dist folder for their siblings, as prescribed in package.json.  If we had something like typescript:main, we could skip this step (microsoft/TypeScript#12561).
- Fixed all the errors the linter complained about.  Since a few packages were written before the linter existed, there were conflicts between their style and its.

Reviewers: O3 Material JavaScript platform reviewers, #material_motion, O2 Material Motion, markwei

Reviewed By: #material_motion, O2 Material Motion, markwei

Tags: #material_motion

Differential Revision: http://codereview.cc/D2395

.travis.yml

@@ -19,7 +19,6 @@ addons:
 before_script:
 - yarn run bootstrap
 script:
-- yarn run test -- --single-run
-- yarn run build
+- yarn run ci
 group: stable
 os: linux

package.json

@@ -6,7 +6,8 @@
     "bootstrap": "yarn; $( yarn bin )/lerna bootstrap",
     "build": "$( yarn bin )/lerna run build",
     "lint": "$( yarn bin)/lerna run lint",
-    "test": "$( yarn bin )/karma start"
+    "test": "$( yarn bin )/karma start",
+    "ci": "yarn run lint; yarn run build; yarn run test -- --single-run"
   },
   "devDependencies": {
     "@types/chai": "^3.4.34",

packages/demos-react/src/map-scroll-to-css/MapScrollPositionToCSSValuePerformer.ts

@@ -15,10 +15,10 @@
  */
 
 import {
-  Plan,
   Performing,
-  PerformingArgs,
   PerformingAddPlanArgs,
+  PerformingArgs,
+  Plan,
 } from 'material-motion-runtime';
 
 import MapScrollPositionToCSSValuePlan from './MapScrollPositionToCSSValuePlan';
@@ -37,7 +37,7 @@ export default class MapScrollPositionToCSSValuePerformer implements Performing
 
     if (!willChange.includes('transform')) {
       if (willChange) {
-        willChange += ', '
+        willChange += ', ';
       }
 
       willChange += 'transform';

packages/demos-react/src/map-scroll-to-css/components/ParallaxClouds.tsx

@@ -34,7 +34,7 @@ export default class ParallaxClouds extends React.Component {
   makeRefForLayer(i) {
     return (element) => {
       this._layers[i] = element;
-    }
+    };
   }
 
   // We have no props and no state, so no need to re-render

packages/demos-react/src/map-scroll-to-css/utils.ts

@@ -15,7 +15,7 @@
  */
 
 
-export function range({ start = 0, end = 1 }:{ start?: number, end?: number } = {}):Array<number> {
+export function range({ start = 0, end = 1 }:{ start?: number, end?: number } = {}):number[] {
   const length = end - start;
   return Array(length + 1).fill().map((_, i) => start + i);
 }

packages/demos-react/src/mount.tsx

@@ -25,5 +25,5 @@ ReactDOM.render(
 );
 
 if (module.hot) {
-  module.hot.accept()
+  module.hot.accept();
 }

packages/demos-react/src/routes.tsx

@@ -16,10 +16,10 @@
 
 import * as React from 'react';
 
-import Link from 'react-router/Link'
-import Match from 'react-router/Match'
-import Miss from 'react-router/Miss'
-import Router from 'react-router/BrowserRouter'
+import Link from 'react-router/Link';
+import Match from 'react-router/Match';
+import Miss from 'react-router/Miss';
+import Router from 'react-router/BrowserRouter';
 
 import ParallaxClouds from './map-scroll-to-css/components/ParallaxClouds';
 
@@ -31,7 +31,7 @@ const links = [
     name: 'MapScrollPositionToCSSValue',
     component: ParallaxClouds,
   }
-]
+];
 
 function Links() {
   return (

packages/runtime/src/Runtime.ts

@@ -65,7 +65,7 @@ export default class Runtime {
     }
 
     const isActiveTokenGenerator = this._isActiveTokenGenerator;
-    const PerformerType: PerformingConstructor = plan._PerformerType;
+    const PerformerType: PerformingConstructor = plan._PerformerType; // tslint:disable-line
 
     const performerMapKey = this._performerMapSelector({ PerformerType, target });
     let performer: Performing;
@@ -85,7 +85,7 @@ export default class Runtime {
       // them, it should ensure we get type errors if a feature isn't threaded
       // through correctly.
 
-      const PerformerOfAllFeatures = PerformerType as PerformingWithAllFeaturesConstructor;
+      const PerformerOfAllFeatures = PerformerType as PerformingWithAllFeaturesConstructor;  // tslint:disable-line
       performer = new PerformerOfAllFeatures({ target, isActiveTokenGenerator });
 
       this._performerMap.set(performerMapKey, performer);

packages/runtime/src/TokenGenerator.ts

@@ -55,7 +55,7 @@ export default class TokenGenerator {
           this._updateTokenCount(-1);
         }
       }
-    }
+    };
   }
 
   _updateTokenCount(delta: number): void {

packages/runtime/src/internal/makeCompoundKeySelector.ts

@@ -16,7 +16,7 @@
 
 export type Dict = {
   [index:string]: any,
-}
+};
 
 /**
  *  A compound key selector searches the object it receives for the values at
@@ -34,8 +34,9 @@ export default function makeCompoundKeySelector(key1:string, key2:string):(dict:
     const value1 = dict[key1];
     const value2 = dict[key2];
 
-    if (!keyMap.has(value1))
+    if (!keyMap.has(value1)) {
       keyMap.set(value1, new Map());
+    }
 
     const value1Map = keyMap.get(value1);
 

packages/streams-dom/src/createMotionElementFromDOMNode.ts

@@ -37,5 +37,5 @@ export default function createMotionElementFromDOMNode(domNode: Element): Motion
       );
     },
 
-  }
+  };
 }

packages/streams-dom/src/index.ts

@@ -14,7 +14,6 @@
  *  under the License.
  */
 
-export * from './types';
 export * from './createMotionElementFromDOMNode';
 export { default as createMotionElementFromDOMNode } from './createMotionElementFromDOMNode';
 

tslint.json

@@ -27,6 +27,7 @@
     "ordered-imports": [
       true,
       {
+        "import-sources-order": "any",
         "named-imports-order": "lowercase-last"
       }
     ],