[core] 100% remove the prop types

mui · Dec 8, 2018 · a134072 · a134072
1 parent 02ccf5e
commit a134072
Show file tree

Hide file tree

Showing 10 changed files with 30 additions and 12 deletions.
diff --git a/babel.config.js b/babel.config.js
@@ -131,7 +131,7 @@ module.exports = {
         [
           'transform-react-remove-prop-types',
           {
-            mode: 'wrap',
+            mode: 'unsafe-wrap',
           },
         ],
       ],
@@ -147,7 +147,7 @@ module.exports = {
         [
           'transform-react-remove-prop-types',
           {
-            mode: 'wrap',
+            mode: 'unsafe-wrap',
           },
         ],
       ],
@@ -163,7 +163,7 @@ module.exports = {
         [
           'transform-react-remove-prop-types',
           {
-            mode: 'wrap',
+            mode: 'unsafe-wrap',
           },
         ],
       ],

diff --git a/packages/material-ui-docs/src/NProgressBar/NProgressBar.js b/packages/material-ui-docs/src/NProgressBar/NProgressBar.js
@@ -97,7 +97,9 @@ NProgressBar.propTypes = {
   children: PropTypes.node,
 };
 
-NProgressBar.propTypes = exactProp(NProgressBar.propTypes, 'NProgressBar');
+if (process.env.NODE_ENV !== 'production') {
+  NProgressBar.propTypes = exactProp(NProgressBar.propTypes);
+}
 
 NProgressBar.defaultProps = {
   children: null,

diff --git a/packages/material-ui-styles/src/StylesProvider.js b/packages/material-ui-styles/src/StylesProvider.js
@@ -88,7 +88,9 @@ StylesProvider.propTypes = {
   sheetsRegistry: PropTypes.object,
 };
 
-StylesProvider.propTypes = exactProp(StylesProvider.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  StylesProvider.propTypes = exactProp(StylesProvider.propTypes);
+}
 
 StylesProvider.defaultProps = {
   disableGeneration: false,

diff --git a/packages/material-ui-styles/src/ThemeProvider.js b/packages/material-ui-styles/src/ThemeProvider.js
@@ -71,6 +71,8 @@ ThemeProvider.propTypes = {
   theme: PropTypes.oneOfType([PropTypes.object, PropTypes.func]).isRequired,
 };
 
-ThemeProvider.propTypes = exactProp(ThemeProvider.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  ThemeProvider.propTypes = exactProp(ThemeProvider.propTypes);
+}
 
 export default ThemeProvider;
diff --git a/packages/material-ui/src/CssBaseline/CssBaseline.js b/packages/material-ui/src/CssBaseline/CssBaseline.js
@@ -48,7 +48,9 @@ CssBaseline.propTypes = {
   classes: PropTypes.object.isRequired,
 };
 
-CssBaseline.propTypes = exactProp(CssBaseline.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  CssBaseline.propTypes = exactProp(CssBaseline.propTypes);
+}
 
 CssBaseline.defaultProps = {
   children: null,

diff --git a/packages/material-ui/src/Hidden/HiddenJs.js b/packages/material-ui/src/Hidden/HiddenJs.js
@@ -130,6 +130,8 @@ HiddenJs.propTypes = {
   xsUp: PropTypes.bool,
 };
 
-HiddenJs.propTypes = exactProp(HiddenJs.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  HiddenJs.propTypes = exactProp(HiddenJs.propTypes);
+}
 
 export default withWidth()(HiddenJs);
diff --git a/packages/material-ui/src/NoSsr/NoSsr.js b/packages/material-ui/src/NoSsr/NoSsr.js
@@ -62,7 +62,9 @@ NoSsr.propTypes = {
   fallback: PropTypes.node,
 };
 
-NoSsr.propTypes = exactProp(NoSsr.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  NoSsr.propTypes = exactProp(NoSsr.propTypes);
+}
 
 NoSsr.defaultProps = {
   defer: false,

diff --git a/packages/material-ui/src/Portal/Portal.js b/packages/material-ui/src/Portal/Portal.js
@@ -99,6 +99,8 @@ Portal.defaultProps = {
   disablePortal: false,
 };
 
-Portal.propTypes = exactProp(Portal.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  Portal.propTypes = exactProp(Portal.propTypes);
+}
 
 export default Portal;
diff --git a/packages/material-ui/src/RootRef/RootRef.js b/packages/material-ui/src/RootRef/RootRef.js
@@ -75,6 +75,8 @@ RootRef.propTypes = {
   rootRef: PropTypes.oneOfType([PropTypes.func, PropTypes.object]).isRequired,
 };
 
-RootRef.propTypes = exactProp(RootRef.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  RootRef.propTypes = exactProp(RootRef.propTypes);
+}
 
 export default RootRef;
diff --git a/packages/material-ui/src/styles/MuiThemeProvider.js b/packages/material-ui/src/styles/MuiThemeProvider.js
@@ -139,7 +139,9 @@ MuiThemeProviderOld.propTypes = {
   theme: PropTypes.oneOfType([PropTypes.object, PropTypes.func]).isRequired,
 };
 
-MuiThemeProviderOld.propTypes = exactProp(MuiThemeProviderOld.propTypes);
+if (process.env.NODE_ENV !== 'production') {
+  MuiThemeProviderOld.propTypes = exactProp(MuiThemeProviderOld.propTypes);
+}
 
 MuiThemeProviderOld.childContextTypes = {
   ...themeListener.contextTypes,