remove prop-types in production build (reduxjs#943)

Author: cellog

.babelrc

@@ -37,6 +37,17 @@
       "plugins": [
         "external-helpers"
       ]
+    },
+    "rollup-production": {
+      "plugins": [
+        "external-helpers",
+        ["transform-react-remove-prop-types", { "removeImport": true }]
+      ]
+    },
+    "production": {
+      "plugins": [
+        ["transform-react-remove-prop-types", { "removeImport": true }]
+      ]
     }
   }
 }

package-lock.json

@@ -30,7 +30,7 @@
     "build:commonjs": "cross-env BABEL_ENV=commonjs babel src --out-dir lib",
     "build:es": "cross-env BABEL_ENV=es babel src --out-dir es",
     "build:umd": "cross-env BABEL_ENV=rollup NODE_ENV=development rollup -c -o dist/react-redux.js",
-    "build:umd:min": "cross-env BABEL_ENV=rollup NODE_ENV=production rollup -c -o dist/react-redux.min.js",
+    "build:umd:min": "cross-env BABEL_ENV=rollup-production NODE_ENV=production rollup -c -o dist/react-redux.min.js",
     "build": "npm run build:commonjs && npm run build:es && npm run build:umd && npm run build:umd:min",
     "clean": "rimraf lib dist es coverage",
     "lint": "eslint src test/utils test/components",
@@ -46,7 +46,6 @@
     "hoist-non-react-statics": "^2.5.5",
     "invariant": "^2.2.4",
     "loose-envify": "^1.1.0",
-    "prop-types": "^15.6.1",
     "react-is": "^16.4.2",
     "shallow-equals": "^1.0.0"
   },
@@ -78,6 +77,7 @@
     "babel-plugin-transform-object-rest-spread": "^6.26.0",
     "babel-plugin-transform-react-display-name": "^6.4.0",
     "babel-plugin-transform-react-jsx": "^6.4.0",
+    "babel-plugin-transform-react-remove-prop-types": "^0.4.14",
     "codecov": "^3.0.2",
     "create-react-class": "^15.6.3",
     "cross-env": "^5.2.0",
@@ -90,6 +90,7 @@
     "jest": "^23.4.1",
     "jest-dom": "^1.12.0",
     "npm-run": "^5.0.1",
+    "prop-types": "^15.6.2",
     "react": "^16.4.2",
     "react-dom": "^16.4.2",
     "react-testing-library": "^5.0.0",

package.json

@@ -28,7 +28,6 @@ const config = {
     commonjs({
       namedExports: {
         'node_modules/react-is/index.js': ['isValidElementType'],
-        'node_modules/prop-types/index.js': ['oneOfType', 'func', 'object']
       }
     })
   ]

rollup.config.js

@@ -1,7 +1,6 @@
 import React, { Component } from 'react'
 import PropTypes from 'prop-types'
 import { Provider as ContextProvider } from './Context'
-import { storeShape } from '../utils/PropTypes'
 
 class Provider extends Component {
   constructor(props) {
@@ -69,7 +68,11 @@ class Provider extends Component {
 }
 
 Provider.propTypes = {
-  store: storeShape.isRequired,
+  store: PropTypes.shape({
+    subscribe: PropTypes.func.isRequired,
+    dispatch: PropTypes.func.isRequired,
+    getState: PropTypes.func.isRequired
+  }),
   context: PropTypes.object,
   children: PropTypes.any
 }

src/components/Provider.js

@@ -1,7 +1,7 @@
 import hoistStatics from 'hoist-non-react-statics'
 import invariant from 'invariant'
 import React, { Component, PureComponent } from 'react'
-import * as propTypes from 'prop-types'
+import propTypes from 'prop-types'
 import shallowEqual from 'shallow-equals'
 import { isValidElementType } from 'react-is'