Remove Flow support in React preset (#6118)

packages/babel-preset-react/README.md

@@ -5,7 +5,6 @@
 This preset always includes the following plugins:
 
 - [syntax-jsx](https://babeljs.io/docs/plugins/syntax-jsx/)
-- [transform-flow-strip-types](https://babeljs.io/docs/plugins/transform-flow-strip-types/)
 - [transform-react-jsx](https://babeljs.io/docs/plugins/transform-react-jsx/)
 - [transform-react-display-name](https://babeljs.io/docs/plugins/transform-react-display-name/)
 
@@ -14,15 +13,7 @@ And with the `development` option:
 - [transform-react-jsx-self](https://babeljs.io/docs/plugins/transform-react-jsx-self/)
 - [transform-react-jsx-source](https://babeljs.io/docs/plugins/transform-react-jsx-source/)
 
-Note: This preset sets the `requireDirective` option on
-`transform-flow-strip-types`. This means Flow annotations and declarations
-will _only_ be removed in files that have a `// @flow ` directive. It will also
-throw errors for any Flow annotations found in files without the directive.
-
-```js
-// @flow
-function foo(numVal: number, strVal: string) {}
-```
+> Note: Flow syntax support is no longer enabled in v7. For that, you will need to add the [Flow preset](https://babeljs.io/docs/plugins/preset-flow/).
 
 ## Install
 

packages/babel-preset-react/package.json

@@ -9,7 +9,6 @@
   "main": "lib/index.js",
   "dependencies": {
     "babel-plugin-syntax-jsx": "7.0.0-alpha.19",
-    "babel-plugin-transform-flow-strip-types": "7.0.0-alpha.19",
     "babel-plugin-transform-react-display-name": "7.0.0-alpha.19",
     "babel-plugin-transform-react-jsx": "7.0.0-alpha.19",
     "babel-plugin-transform-react-jsx-self": "7.0.0-alpha.19",

packages/babel-preset-react/src/index.js

@@ -1,4 +1,3 @@
-import transformFlowStripTypes from "babel-plugin-transform-flow-strip-types";
 import transformReactJSX from "babel-plugin-transform-react-jsx";
 import transformSyntaxJSX from "babel-plugin-syntax-jsx";
 import transformReactDisplayName from "babel-plugin-transform-react-display-name";
@@ -20,7 +19,6 @@ export default function(context, opts = {}) {
 
       development && transformReactJSXSource,
       development && transformReactJSXSelf,
-      [transformFlowStripTypes, { requireDirective: true }],
     ].filter(Boolean),
   };
 }

packages/babel-preset-react/test/fixtures/flow/options.json

@@ -1,5 +0,0 @@
-{
-  "presets": [
-    "react"
-  ]
-}

packages/babel-preset-react/test/fixtures/flow/strip-with-directive/actual.js

@@ -1,2 +0,0 @@
-// @flow
-function foo(numVal: number, strVal: string) {}

packages/babel-preset-react/test/fixtures/flow/strip-with-directive/expected.js

@@ -1 +0,0 @@
-function foo(numVal, strVal) {}

packages/babel-preset-react/test/fixtures/flow/throws-without-directive/actual.js

@@ -1 +0,0 @@
-function foo(numVal: number, strVal: string) {}

packages/babel-preset-react/test/fixtures/flow/throws-without-directive/expected.js

@@ -1 +0,0 @@
-function foo(numVal: number, strVal: string) {}

packages/babel-preset-react/test/fixtures/flow/throws-without-directive/options.json

@@ -1,3 +0,0 @@
-{
-  "throws": "A @flow directive is required when using Flow annotations with babel-preset-react or the `requireDirective` option."
-}