Rerun prettier and pin version (#3058)

* rerun prettier for all files

* pin prettier to 1.6.1

package.json

@@ -19,7 +19,7 @@
     "lerna": "^2.0.0",
     "lerna-changelog": "^0.6.0",
     "lint-staged": "^3.3.1",
-    "prettier": "^1.5.2"
+    "prettier": "1.6.1"
   },
   "lint-staged": {
     "*.js": [

packages/create-react-app/createReactApp.js

@@ -455,7 +455,9 @@ function checkNpmVersion() {
   let hasMinNpm = false;
   let npmVersion = null;
   try {
-    npmVersion = execSync('npm --version').toString().trim();
+    npmVersion = execSync('npm --version')
+      .toString()
+      .trim();
     hasMinNpm = semver.gte(npmVersion, '3.0.0');
   } catch (err) {
     // ignore

packages/react-dev-utils/launchEditor.js

@@ -43,7 +43,7 @@ const COMMON_EDITORS_OSX = {
   '/Applications/CLion.app/Contents/MacOS/clion':
     '/Applications/CLion.app/Contents/MacOS/clion',
   '/Applications/IntelliJ IDEA.app/Contents/MacOS/idea':
-      '/Applications/IntelliJ IDEA.app/Contents/MacOS/idea',
+    '/Applications/IntelliJ IDEA.app/Contents/MacOS/idea',
   '/Applications/PhpStorm.app/Contents/MacOS/phpstorm':
     '/Applications/PhpStorm.app/Contents/MacOS/phpstorm',
   '/Applications/PyCharm.app/Contents/MacOS/pycharm':
@@ -53,7 +53,7 @@ const COMMON_EDITORS_OSX = {
   '/Applications/RubyMine.app/Contents/MacOS/rubymine':
     '/Applications/RubyMine.app/Contents/MacOS/rubymine',
   '/Applications/WebStorm.app/Contents/MacOS/webstorm':
-      '/Applications/WebStorm.app/Contents/MacOS/webstorm',
+    '/Applications/WebStorm.app/Contents/MacOS/webstorm',
 };
 
 const COMMON_EDITORS_WIN = [

packages/react-error-overlay/src/components/Header.js

@@ -29,11 +29,7 @@ type HeaderPropType = {|
 |};
 
 function Header(props: HeaderPropType) {
-  return (
-    <div style={headerStyle}>
-      {props.headerText}
-    </div>
-  );
+  return <div style={headerStyle}>{props.headerText}</div>;
 }
 
 export default Header;

packages/react-error-overlay/src/containers/RuntimeErrorContainer.js

@@ -54,13 +54,14 @@ class RuntimeErrorContainer extends PureComponent {
     return (
       <ErrorOverlay shortcutHandler={this.shortcutHandler}>
         <CloseButton close={close} />
-        {totalErrors > 1 &&
+        {totalErrors > 1 && (
           <NavigationBar
             currentError={this.state.currentIndex + 1}
             totalErrors={totalErrors}
             previous={this.previous}
             next={this.next}
-          />}
+          />
+        )}
         <RuntimeError
           errorRecord={errorRecords[this.state.currentIndex]}
           launchEditorEndpoint={this.props.launchEditorEndpoint}

packages/react-error-overlay/src/containers/StackFrame.js

@@ -155,9 +155,7 @@ class StackFrame extends Component {
     const canOpenInEditor = this.canOpenInEditor();
     return (
       <div>
-        <div>
-          {functionName}
-        </div>
+        <div>{functionName}</div>
         <div style={linkStyle}>
           <a
             style={canOpenInEditor ? anchorStyle : null}
@@ -168,7 +166,7 @@ class StackFrame extends Component {
             {url}
           </a>
         </div>
-        {codeBlockProps &&
+        {codeBlockProps && (
           <span>
             <a
               onClick={canOpenInEditor ? this.openInEditor : null}
@@ -179,7 +177,8 @@ class StackFrame extends Component {
             <button style={toggleStyle} onClick={this.toggleCompiled}>
               {'View ' + (compiled ? 'source' : 'compiled')}
             </button>
-          </span>}
+          </span>
+        )}
       </div>
     );
   }

packages/react-error-overlay/src/containers/StackTrace.js

@@ -84,11 +84,7 @@ class StackTrace extends Component {
   }
 
   render() {
-    return (
-      <div style={traceStyle}>
-        {this.renderFrames()}
-      </div>
-    );
+    return <div style={traceStyle}>{this.renderFrames()}</div>;
   }
 }
 

packages/react-error-overlay/src/utils/parser.js

@@ -13,13 +13,16 @@ import StackFrame from './stack-frame';
 const regexExtractLocation = /\(?(.+?)(?::(\d+))?(?::(\d+))?\)?$/;
 
 function extractLocation(token: string): [string, number, number] {
-  return regexExtractLocation.exec(token).slice(1).map(v => {
-    const p = Number(v);
-    if (!isNaN(p)) {
-      return p;
-    }
-    return v;
-  });
+  return regexExtractLocation
+    .exec(token)
+    .slice(1)
+    .map(v => {
+      const p = Number(v);
+      if (!isNaN(p)) {
+        return p;
+      }
+      return v;
+    });
 }
 
 const regexValidFrame_Chrome = /^\s*(at|in)\s.+(:\d+)/;
@@ -55,7 +58,10 @@ function parseStack(stack: string[]): StackFrame[] {
         if (e.indexOf('(at ') !== -1) {
           e = e.replace(/\(at /, '(');
         }
-        const data = e.trim().split(/\s+/g).slice(1);
+        const data = e
+          .trim()
+          .split(/\s+/g)
+          .slice(1);
         const last = data.pop();
         return new StackFrame(data.join(' ') || null, ...extractLocation(last));
       }

packages/react-error-overlay/src/utils/warnings.js

@@ -11,7 +11,10 @@
 import type { ReactFrame } from '../effects/proxyConsole';
 
 function stripInlineStacktrace(message: string): string {
-  return message.split('\n').filter(line => !line.match(/^\s*in/)).join('\n'); // "  in Foo"
+  return message
+    .split('\n')
+    .filter(line => !line.match(/^\s*in/))
+    .join('\n'); // "  in Foo"
 }
 
 function massage(

packages/react-scripts/fixtures/kitchensink/src/features/env/FileEnvVariables.js

@@ -9,7 +9,7 @@
 
 import React from 'react';
 
-export default () =>
+export default () => (
   <span>
     <span id="feature-file-env-original-1">
       {process.env.REACT_APP_ORIGINAL_1}
@@ -21,7 +21,6 @@ export default () =>
       {process.env.REACT_APP_DEVELOPMENT}
       {process.env.REACT_APP_PRODUCTION}
     </span>
-    <span id="feature-file-env-x">
-      {process.env.REACT_APP_X}
-    </span>
-  </span>;
+    <span id="feature-file-env-x">{process.env.REACT_APP_X}</span>
+  </span>
+);

packages/react-scripts/fixtures/kitchensink/src/features/env/NodePath.js

@@ -33,11 +33,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-node-path">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/env/PublicUrl.js

@@ -9,7 +9,6 @@
 
 import React from 'react';
 
-export default () =>
-  <span id="feature-public-url">
-    {process.env.PUBLIC_URL}.
-  </span>;
+export default () => (
+  <span id="feature-public-url">{process.env.PUBLIC_URL}.</span>
+);

packages/react-scripts/fixtures/kitchensink/src/features/env/ShellEnvVariables.js

@@ -9,7 +9,8 @@
 
 import React from 'react';
 
-export default () =>
+export default () => (
   <span id="feature-shell-env-variables">
     {process.env.REACT_APP_SHELL_ENV_MESSAGE}.
-  </span>;
+  </span>
+);

packages/react-scripts/fixtures/kitchensink/src/features/syntax/ArrayDestructuring.js

@@ -38,11 +38,7 @@ export default class extends Component {
       <div id="feature-array-destructuring">
         {this.state.users.map(user => {
           const [id, name] = user;
-          return (
-            <div key={id}>
-              {name}
-            </div>
-          );
+          return <div key={id}>{name}</div>;
         })}
       </div>
     );

packages/react-scripts/fixtures/kitchensink/src/features/syntax/ArraySpread.js

@@ -41,11 +41,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-array-spread">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/AsyncAwait.js

@@ -41,11 +41,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-async-await">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/ClassProperties.js

@@ -29,11 +29,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-class-properties">
-        {this.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/ComputedProperties.js

@@ -41,11 +41,9 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-computed-properties">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.user_name}
-          </div>
-        )}
+        {this.state.users.map(user => (
+          <div key={user.id}>{user.user_name}</div>
+        ))}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/CustomInterpolation.js

@@ -53,11 +53,11 @@ export default class extends Component {
 
     return (
       <div id="feature-custom-interpolation">
-        {this.state.users.map(user =>
+        {this.state.users.map(user => (
           <div key={user.id} style={veryInlineStyle}>
             {user.name}
           </div>
-        )}
+        ))}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/DefaultParameters.js

@@ -41,11 +41,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-default-parameters">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/DestructuringAndAwait.js

@@ -43,11 +43,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-destructuring-and-await">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/Generators.js

@@ -43,11 +43,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-generators">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/ObjectDestructuring.js

@@ -43,11 +43,7 @@ export default class extends Component {
       <div id="feature-object-destructuring">
         {this.state.users.map(user => {
           const { id, name } = user;
-          return (
-            <div key={id}>
-              {name}
-            </div>
-          );
+          return <div key={id}>{name}</div>;
         })}
       </div>
     );

packages/react-scripts/fixtures/kitchensink/src/features/syntax/ObjectSpread.js

@@ -41,11 +41,11 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-object-spread">
-        {this.state.users.map(user =>
+        {this.state.users.map(user => (
           <div key={user.id}>
             {user.name}: {user.age}
           </div>
-        )}
+        ))}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/Promises.js

@@ -42,11 +42,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-promises">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/RestAndDefault.js

@@ -41,11 +41,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-rest-and-default">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/RestParameters.js

@@ -41,11 +41,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-rest-parameters">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/syntax/TemplateInterpolation.js

@@ -41,11 +41,7 @@ export default class extends Component {
   render() {
     return (
       <div id="feature-template-interpolation">
-        {this.state.users.map(user =>
-          <div key={user.id}>
-            {user.name}
-          </div>
-        )}
+        {this.state.users.map(user => <div key={user.id}>{user.name}</div>)}
       </div>
     );
   }

packages/react-scripts/fixtures/kitchensink/src/features/webpack/ImageInclusion.js

@@ -10,5 +10,6 @@
 import React from 'react';
 import tiniestCat from './assets/tiniest-cat.jpg';
 
-export default () =>
-  <img id="feature-image-inclusion" src={tiniestCat} alt="tiniest cat" />;
+export default () => (
+  <img id="feature-image-inclusion" src={tiniestCat} alt="tiniest cat" />
+);

packages/react-scripts/fixtures/kitchensink/src/features/webpack/JsonInclusion.js

@@ -10,7 +10,4 @@
 import React from 'react';
 import { abstract } from './assets/abstract.json';
 
-export default () =>
-  <summary id="feature-json-inclusion">
-    {abstract}
-  </summary>;
+export default () => <summary id="feature-json-inclusion">{abstract}</summary>;

packages/react-scripts/fixtures/kitchensink/src/features/webpack/LinkedModules.js

@@ -16,9 +16,5 @@ export default () => {
   if (!test() || v !== '2.0.0') {
     throw new Error('Functionality test did not pass.');
   }
-  return (
-    <p id="feature-linked-modules">
-      {v}
-    </p>
-  );
+  return <p id="feature-linked-modules">{v}</p>;
 };

packages/react-scripts/fixtures/kitchensink/src/features/webpack/NoExtInclusion.js

@@ -14,7 +14,8 @@ const text = aFileWithoutExt.includes('base64')
   ? atob(aFileWithoutExt.split('base64,')[1]).trim()
   : aFileWithoutExt;
 
-export default () =>
+export default () => (
   <a id="feature-no-ext-inclusion" href={text}>
     aFileWithoutExt
-  </a>;
+  </a>
+);

packages/react-scripts/fixtures/kitchensink/src/features/webpack/UnknownExtInclusion.js

@@ -14,7 +14,8 @@ const text = aFileWithExtUnknown.includes('base64')
   ? atob(aFileWithExtUnknown.split('base64,')[1]).trim()
   : aFileWithExtUnknown;
 
-export default () =>
+export default () => (
   <a id="feature-unknown-ext-inclusion" href={text}>
     aFileWithExtUnknown
-  </a>;
+  </a>
+);

packages/react-scripts/scripts/eject.js

@@ -167,9 +167,11 @@ inquirer
     // Sort the deps
     const unsortedDependencies = appPackage.dependencies;
     appPackage.dependencies = {};
-    Object.keys(unsortedDependencies).sort().forEach(key => {
-      appPackage.dependencies[key] = unsortedDependencies[key];
-    });
+    Object.keys(unsortedDependencies)
+      .sort()
+      .forEach(key => {
+        appPackage.dependencies[key] = unsortedDependencies[key];
+      });
     console.log();
 
     console.log(cyan('Updating the scripts'));