Merge pull request #1201 from cpojer/master

Add commas to multiline function calls.

Author: sampepose

.eslintrc

@@ -12,6 +12,8 @@
     "array-bracket-spacing": [2, "never"],
     "arrow-spacing": [2],
     "babel/arrow-parens": [2, "as-needed"],
+    "babel/flow-object-type": [2, "comma"],
+    "babel/func-params-comma-dangle": [2, "always-multiline"],
     "brace-style": [2, "1tbs", {"allowSingleLine": true}],
     "comma-dangle": [2, "always-multiline"],
     "comma-spacing": [2],

examples/.eslintrc

@@ -0,0 +1,5 @@
+{
+  "rules": {
+    "babel/func-params-comma-dangle": 0
+  }
+}

integration_tests/.eslintrc

@@ -0,0 +1,5 @@
+{
+  "rules": {
+    "babel/func-params-comma-dangle": 0
+  }
+}

lerna.json

@@ -1,6 +1,6 @@
 {
   "lerna": "2.0.0-beta.20",
-  "version": "12.1.2",
+  "version": "12.1.3",
   "linkedFiles": {
     "prefix": "/**\n * @flow\n */\n"
   }

package.json

@@ -7,7 +7,7 @@
     "babel-plugin-transform-flow-strip-types": "^6.8.0",
     "chalk": "^1.1.3",
     "eslint": "^2.13.1",
-    "eslint-plugin-babel": "^3.2.0",
+    "eslint-plugin-babel": "^3.3.0",
     "fbjs-scripts": "^0.7.1",
     "flow-bin": "^0.27.0",
     "glob": "^7.0.4",

packages/babel-plugin-jest-hoist/src/index.js

@@ -81,7 +81,7 @@ FUNCTIONS.mock = args => {
     const moduleFactory = args[1];
     invariant(
       moduleFactory.isFunction(),
-      'The second argument of `jest.mock` must be a function.'
+      'The second argument of `jest.mock` must be a function.',
     );
 
     const ids = new Set();
@@ -108,7 +108,7 @@ FUNCTIONS.mock = args => {
           'reference any outside variables.\n' +
           'Invalid variable access: ' + name + '\n' +
           'Whitelisted objects: ' +
-          Object.keys(WHITELISTED_IDENTIFIERS).join(', ') + '.'
+          Object.keys(WHITELISTED_IDENTIFIERS).join(', ') + '.',
         );
       }
     }

packages/jest-changed-files/src/__tests__/hg-test.js

@@ -35,7 +35,7 @@ describe('hgChecker', () => {
     it('returns null for non hg repo folder', () =>
       hg.isHGRepository(tmpdir).then(res => {
         expect(res).toBeNull();
-      })
+      }),
     );
 
     it('returns dirname for hg repo folder', () => {
@@ -56,7 +56,7 @@ describe('hgChecker', () => {
     it('returns an empty array for hg repo folder without modified files', () =>
       hg.findChangedFiles(tmpdir).then(res => {
         expect(res).toEqual([]);
-      })
+      }),
     );
 
     it('returns an array of modified files for hg repo folder', () => {

packages/jest-changed-files/src/git.js

@@ -32,7 +32,7 @@ function findChangedFiles(cwd: string): Promise<Array<Path>> {
           resolve([]);
         } else {
           resolve(stdout.split('\n').map(
-            changedPath => path.resolve(cwd, changedPath)
+            changedPath => path.resolve(cwd, changedPath),
           ));
         }
       } else {

packages/jest-changed-files/src/hg.js

@@ -42,7 +42,7 @@ function findChangedFiles(cwd: string, options: Options): Promise<Array<Path>> {
           resolve([]);
         } else {
           resolve(stdout.split('\n').map(
-            changedPath => path.resolve(cwd, changedPath)
+            changedPath => path.resolve(cwd, changedPath),
           ));
         }
       } else {

packages/jest-cli/src/CoverageCollector.js

@@ -18,7 +18,7 @@ let _memoizedCoverageTemplate = null;
 function _getCoverageTemplate() {
   if (_memoizedCoverageTemplate === null) {
     _memoizedCoverageTemplate = require('lodash.template')(
-      fs.readFileSync(COVERAGE_TEMPLATE_PATH, 'utf8')
+      fs.readFileSync(COVERAGE_TEMPLATE_PATH, 'utf8'),
     );
   }
   return _memoizedCoverageTemplate;
@@ -67,7 +67,7 @@ class CoverageCollector {
     for (nodeIndex in instrumentationInfo.nodes) {
       if (!this._coverageDataStore.nodes.hasOwnProperty(nodeIndex)) {
         coverageInfo.uncoveredSpans.push(
-          instrumentationInfo.nodes[nodeIndex].loc
+          instrumentationInfo.nodes[nodeIndex].loc,
         );
       }
     }