Run prettier over the codebase to reduce CI problems.

scripts/jest/test-framework-setup.js

@@ -36,9 +36,10 @@ env.beforeEach(() => {
         compare(actual) {
           return {
             pass: callCount === 0,
-            message: 'Expected test not to warn. If the warning is expected, mock ' +
-              "it out using spyOn(console, 'error'); and test that the " +
-              'warning occurs.',
+            message:
+              'Expected test not to warn. If the warning is expected, mock ' +
+                "it out using spyOn(console, 'error'); and test that the " +
+                'warning occurs.',
           };
         },
       };

scripts/release-manager/commands/init.js

@@ -27,7 +27,9 @@ module.exports = function(vorpal, app) {
             {
               name: 'githubToken',
               type: 'input',
-              message: `${chalk.bold('GitHub token?')} ${chalk.grey('(needs "repo" privs)')} `,
+              message: `${chalk.bold('GitHub token?')} ${chalk.grey(
+                '(needs "repo" privs)'
+              )} `,
             },
             {
               name: 'reactPath',

scripts/release-manager/commands/npm-check-access.js

@@ -27,7 +27,9 @@ module.exports = function(vorpal, app) {
 
         if (packagesNeedingAccess.length) {
           this.log(
-            `${chalk.red('FAILED')} You don't have access to all of the packages ` +
+            `${chalk.red(
+              'FAILED'
+            )} You don't have access to all of the packages ` +
               `you need. We just opened a URL to file a new issue requesting access.`
           );
           opn(

scripts/release-manager/commands/npm-grant-access.js

@@ -27,11 +27,15 @@ module.exports = function(vorpal, app) {
 
           if (packagesNeedingAccess.length) {
             this.log(
-              `${chalk.yellow('PENDING')} Granting access to ${packagesNeedingAccess}`
+              `${chalk.yellow(
+                'PENDING'
+              )} Granting access to ${packagesNeedingAccess}`
             );
             npmUtils.grantAccess(app, answers.username, packagesNeedingAccess);
             this.log(
-              `${chalk.green('OK')} Access has been granted to ${answers.username}.`
+              `${chalk.green(
+                'OK'
+              )} Access has been granted to ${answers.username}.`
             );
             resolve();
           } else {

scripts/release-manager/commands/npm-publish.js

@@ -28,7 +28,8 @@ module.exports = function(vorpal, app) {
         this.prompt([
           {
             type: 'confirm',
-            message: 'Did you run `grunt build` or `grunt release` and bump the version number?',
+            message:
+              'Did you run `grunt build` or `grunt release` and bump the version number?',
             default: false,
             name: 'checklist',
           },

scripts/release-manager/commands/stable-prs.js

@@ -66,13 +66,15 @@ module.exports = function(vorpal, app) {
               {
                 name: 'destMilestone',
                 type: 'list',
-                message: 'Which milestone should we assign PRs to upon completion?',
+                message:
+                  'Which milestone should we assign PRs to upon completion?',
                 choices: milestoneChoices,
               },
               {
                 name: 'labels',
                 type: 'checkbox',
-                message: 'Which PRs should we select (use spacebar to check all that apply)',
+                message:
+                  'Which PRs should we select (use spacebar to check all that apply)',
                 choices: labelChoices,
               },
             ]).then(answers => {
@@ -136,7 +138,9 @@ module.exports = function(vorpal, app) {
                       .filter(pr => {
                         if (!pr.merged_at) {
                           this.log(
-                            `${chalk.yellow.bold('WARNING')} ${pr.html_url} was not merged,` +
+                            `${chalk.yellow.bold(
+                              'WARNING'
+                            )} ${pr.html_url} was not merged,` +
                               ` should have the milestone unset.`
                           );
                           return false;
@@ -180,7 +184,9 @@ module.exports = function(vorpal, app) {
                 })
                 .catch(err => {
                   this.log(
-                    `${chalk.red.bold('ERROR')} Something went wrong and your repo is` +
+                    `${chalk.red.bold(
+                      'ERROR'
+                    )} Something went wrong and your repo is` +
                       ` probably in a bad state. Sorry.`
                   );
                   resolve({
@@ -243,13 +249,16 @@ function cherryPickPRs(app, prs) {
           return this.prompt({
             name: 'handle',
             type: 'list',
-            message: `${chalk.red`Failed!`} ${chalk.yellow('This must be resolved manually!')}`,
+            message: `${chalk.red`Failed!`} ${chalk.yellow(
+              'This must be resolved manually!'
+            )}`,
             choices: [
               {value: 'ok', name: 'Continue, mark successful'},
               {value: 'skip', name: 'Continue, mark skipped'},
               {
                 value: 'abort',
-                name: 'Abort process. Will require manual resetting of git state.',
+                name:
+                  'Abort process. Will require manual resetting of git state.',
               },
             ],
           }).then(answers => {

scripts/release-manager/commands/utils/npm.js

@@ -26,7 +26,9 @@ function generateAccessNeededIssue(username, packages) {
     body: `In order to publish React to npm I need access to the following repositories:
 ${packages.map(pkg => `- [${pkg}](https://npm.im/${pkg})`).join('\n')}`,
   };
-  return `https://github.com/facebook/react/issues/new?${querystring.stringify(data)}`;
+  return `https://github.com/facebook/react/issues/new?${querystring.stringify(
+    data
+  )}`;
 }
 
 function grantAccess(app, username, packages) {

scripts/release-manager/commands/version.js

@@ -158,7 +158,8 @@ module.exports = function(vorpal, app) {
           {
             name: 'tag',
             type: 'confirm',
-            message: 'Tag the version commit (not necessary for non-stable releases)?',
+            message:
+              'Tag the version commit (not necessary for non-stable releases)?',
             default: true,
             when: res => res.commit,
           },

scripts/rollup/modules.js

@@ -225,7 +225,9 @@ function replaceFbjsModuleAliases(bundleType) {
   }
 }
 
-const devOnlyModuleStub = `'${resolve('./scripts/rollup/shims/rollup/DevOnlyStubShim.js')}'`;
+const devOnlyModuleStub = `'${resolve(
+  './scripts/rollup/shims/rollup/DevOnlyStubShim.js'
+)}'`;
 
 function replaceDevOnlyStubbedModules(bundleType) {
   switch (bundleType) {

src/isomorphic/classic/element/ReactElementValidator.js

@@ -112,7 +112,9 @@ function validateExplicitKey(element, parentType) {
     element._owner !== ReactCurrentOwner.current
   ) {
     // Give the component that originally created this child.
-    childOwner = ` It was passed a child from ${getComponentName(element._owner)}.`;
+    childOwner = ` It was passed a child from ${getComponentName(
+      element._owner,
+    )}.`;
   }
 
   warning(

src/renderers/__tests__/ReactCompositeComponentState-test.js

@@ -414,7 +414,8 @@ describe('ReactCompositeComponent-state', () => {
         this.setState({step: 2}, () => {
           // Tests that earlier setState callbacks are not dropped
           ops.push(
-            `callback -- step: ${this.state.step}, extra: ${!!this.state.extra}`,
+            `callback -- step: ${this.state.step}, extra: ${!!this.state
+              .extra}`,
           );
         });
         // Treat like replaceState
@@ -458,7 +459,8 @@ describe('ReactCompositeComponent-state', () => {
           this.setState({step: 2}, () => {
             // Tests that earlier setState callbacks are not dropped
             ops.push(
-              `callback -- step: ${this.state.step}, extra: ${!!this.state.extra}`,
+              `callback -- step: ${this.state.step}, extra: ${!!this.state
+                .extra}`,
             );
           });
           // Treat like replaceState

src/renderers/__tests__/ReactErrorBoundaries-test.js

@@ -2001,11 +2001,10 @@ describe('ReactErrorBoundaries', () => {
       ReactDOM.render(
         <ErrorBoundary>
           <BrokenComponentDidMountErrorBoundary
-            renderError={error => (
+            renderError={error =>
               <div>
                 We should never catch our own error: {error.message}.
-              </div>
-            )}
+              </div>}
           />
         </ErrorBoundary>,
         container,

src/renderers/__tests__/ReactIdentity-test.js

@@ -25,12 +25,11 @@ describe('ReactIdentity', () => {
 
   it('should allow key property to express identity', () => {
     var node;
-    var Component = props => (
+    var Component = props =>
       <div ref={c => (node = c)}>
         <div key={props.swap ? 'banana' : 'apple'} />
         <div key={props.swap ? 'apple' : 'banana'} />
-      </div>
-    );
+      </div>;
 
     var container = document.createElement('div');
     ReactDOM.render(<Component />, container);

src/renderers/dom/ReactDOMStackEntry.js

@@ -61,7 +61,8 @@ if (
 ) {
   __REACT_DEVTOOLS_GLOBAL_HOOK__.inject({
     ComponentTree: {
-      getClosestInstanceFromNode: ReactDOMComponentTree.getClosestInstanceFromNode,
+      getClosestInstanceFromNode:
+        ReactDOMComponentTree.getClosestInstanceFromNode,
       getNodeFromInstance: function(inst) {
         // inst is an internal instance (but could be a composite)
         if (inst._renderedComponent) {

src/renderers/dom/fiber/ReactDOMFiberEntry.js

@@ -96,10 +96,12 @@ let selectionInformation: ?mixed = null;
  * @internal
  */
 function isValidContainer(node) {
-  return !!(node &&
+  return !!(
+    node &&
     (node.nodeType === ELEMENT_NODE ||
       node.nodeType === DOCUMENT_NODE ||
-      node.nodeType === DOCUMENT_FRAGMENT_NODE));
+      node.nodeType === DOCUMENT_FRAGMENT_NODE)
+  );
 }
 
 function getReactRootElementInContainer(container: any) {
@@ -116,9 +118,11 @@ function getReactRootElementInContainer(container: any) {
 
 function shouldReuseContent(container) {
   const rootElement = getReactRootElementInContainer(container);
-  return !!(rootElement &&
+  return !!(
+    rootElement &&
     rootElement.nodeType === ELEMENT_NODE &&
-    rootElement.getAttribute(ID_ATTRIBUTE_NAME));
+    rootElement.getAttribute(ID_ATTRIBUTE_NAME)
+  );
 }
 
 function shouldAutoFocusHostComponent(type: string, props: Props): boolean {
@@ -468,8 +472,9 @@ function renderSubtreeIntoContainer(
   if (__DEV__) {
     const isRootRenderedBySomeReact = !!container._reactRootContainer;
     const rootEl = getReactRootElementInContainer(container);
-    const hasNonRootReactChild = !!(rootEl &&
-      ReactDOMComponentTree.getInstanceFromNode(rootEl));
+    const hasNonRootReactChild = !!(
+      rootEl && ReactDOMComponentTree.getInstanceFromNode(rootEl)
+    );
 
     warning(
       !hasNonRootReactChild || isRootRenderedBySomeReact,

src/renderers/dom/shared/DOMProperty.js

@@ -163,8 +163,8 @@ var DOMProperty = {
   ROOT_ATTRIBUTE_NAME: 'data-reactroot',
 
   ATTRIBUTE_NAME_START_CHAR: ATTRIBUTE_NAME_START_CHAR,
-  ATTRIBUTE_NAME_CHAR: ATTRIBUTE_NAME_START_CHAR +
-    '\\-.0-9\\u00B7\\u0300-\\u036F\\u203F-\\u2040',
+  ATTRIBUTE_NAME_CHAR:
+    ATTRIBUTE_NAME_START_CHAR + '\\-.0-9\\u00B7\\u0300-\\u036F\\u203F-\\u2040',
 
   /**
    * Map from property "standard name" to an object with info about how to set

src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js

@@ -1667,8 +1667,8 @@ describe('ReactDOMComponent', () => {
       var previousLine = (matches || [])[1];
 
       expectDev(console.error.calls.argsFor(1)[0]).toContain('onClick');
-      matches = console.error.calls.argsFor(1)[0].match(/.*\(.*:(\d+)\).*/) || {
-      };
+      matches =
+        console.error.calls.argsFor(1)[0].match(/.*\(.*:(\d+)\).*/) || {};
       var currentLine = (matches || [])[1];
 
       //verify line number has a proper relative difference,

src/renderers/dom/shared/__tests__/ReactDOMServerIntegration-test.js

@@ -2137,22 +2137,22 @@ describe('ReactDOMServerIntegration', () => {
       it('should error reconnecting a div with children separated by whitespace on the client', () =>
         expectMarkupMismatch(
           <div id="parent"><div id="child1" /><div id="child2" /></div>,
-          // prettier-ignore
-          <div id="parent"><div id="child1" />      <div id="child2" /></div>, // eslint-disable-line no-multi-spaces
+          // eslint-disable-next-line no-multi-spaces
+          <div id="parent"><div id="child1" />      <div id="child2" /></div>,
         ));
 
       it('should error reconnecting a div with children separated by different whitespace on the server', () =>
         expectMarkupMismatch(
-          // prettier-ignore
-          <div id="parent"><div id="child1" />      <div id="child2" /></div>, // eslint-disable-line no-multi-spaces
+          // eslint-disable-next-line no-multi-spaces
+          <div id="parent"><div id="child1" />      <div id="child2" /></div>,
           <div id="parent"><div id="child1" /><div id="child2" /></div>,
         ));
 
       it('should error reconnecting a div with children separated by different whitespace', () =>
         expectMarkupMismatch(
           <div id="parent"><div id="child1" /> <div id="child2" /></div>,
-          // prettier-ignore
-          <div id="parent"><div id="child1" />      <div id="child2" /></div>, // eslint-disable-line no-multi-spaces
+          // eslint-disable-next-line no-multi-spaces
+          <div id="parent"><div id="child1" />      <div id="child2" /></div>,
         ));
 
       it('can distinguish an empty component from a dom node', () =>

src/renderers/dom/shared/eventPlugins/SelectEventPlugin.js

@@ -154,8 +154,8 @@ var SelectEventPlugin = {
     var doc = nativeEventTarget.window === nativeEventTarget
       ? nativeEventTarget.document
       : nativeEventTarget.nodeType === DOCUMENT_NODE
-          ? nativeEventTarget
-          : nativeEventTarget.ownerDocument;
+        ? nativeEventTarget
+        : nativeEventTarget.ownerDocument;
     if (!doc || !isListeningToAllDependencies('onSelect', doc)) {
       return null;
     }

src/renderers/dom/shared/hooks/ReactDOMUnknownPropertyHook.js

@@ -69,8 +69,8 @@ if (__DEV__) {
     var standardName = DOMProperty.isCustomAttribute(lowerCasedName)
       ? lowerCasedName
       : DOMProperty.getPossibleStandardName.hasOwnProperty(lowerCasedName)
-          ? DOMProperty.getPossibleStandardName[lowerCasedName]
-          : null;
+        ? DOMProperty.getPossibleStandardName[lowerCasedName]
+        : null;
 
     var registrationName = EventPluginRegistry.possibleRegistrationNames.hasOwnProperty(
       lowerCasedName,

src/renderers/dom/shared/syntheticEvents/SyntheticWheelEvent.js

@@ -29,9 +29,9 @@ var WheelEventInterface = {
       ? event.deltaY
       : // Fallback to `wheelDeltaY` for Webkit and normalize (down is positive).
         'wheelDeltaY' in event
-          ? -event.wheelDeltaY
-          : // Fallback to `wheelDelta` for IE<9 and normalize (down is positive).
-            'wheelDelta' in event ? -event.wheelDelta : 0;
+        ? -event.wheelDeltaY
+        : // Fallback to `wheelDelta` for IE<9 and normalize (down is positive).
+          'wheelDelta' in event ? -event.wheelDelta : 0;
   },
   deltaZ: null,
 

src/renderers/dom/stack/client/ReactComponentBrowserEnvironment.js

@@ -20,7 +20,8 @@ var ReactDOMIDOperations = require('ReactDOMIDOperations');
  * need for this injection.
  */
 var ReactComponentBrowserEnvironment = {
-  processChildrenUpdates: ReactDOMIDOperations.dangerouslyProcessChildrenUpdates,
+  processChildrenUpdates:
+    ReactDOMIDOperations.dangerouslyProcessChildrenUpdates,
 
   replaceNodeWithMarkup: DOMChildrenOperations.dangerouslyReplaceNodeWithMarkup,
 };

src/renderers/dom/stack/client/ReactMount.js

@@ -205,9 +205,11 @@ function hasNonRootReactChild(container) {
  */
 function nodeIsRenderedByOtherInstance(container) {
   var rootEl = getReactRootElementInContainer(container);
-  return !!(rootEl &&
+  return !!(
+    rootEl &&
     isReactNode(rootEl) &&
-    !ReactDOMComponentTree.getInstanceFromNode(rootEl));
+    !ReactDOMComponentTree.getInstanceFromNode(rootEl)
+  );
 }
 
 /**
@@ -218,10 +220,12 @@ function nodeIsRenderedByOtherInstance(container) {
  * @internal
  */
 function isValidContainer(node) {
-  return !!(node &&
+  return !!(
+    node &&
     (node.nodeType === ELEMENT_NODE ||
       node.nodeType === DOCUMENT_NODE ||
-      node.nodeType === DOCUMENT_FRAGMENT_NODE));
+      node.nodeType === DOCUMENT_FRAGMENT_NODE)
+  );
 }
 
 /**