Merge pull request #639 from dannycolin/remove-prop-types

Remove deprecated prop-types

Author: lotas

package.json

@@ -87,7 +87,6 @@
     "postcss-safe-parser": "5.0.2",
     "prettier": "^2.0.0",
     "prompts": "2.4.2",
-    "prop-types": "^15.6.1",
     "qs": "^6.5.2",
     "ramda": "^0.28.0",
     "react": "18",

src/components/Dashboard/index.jsx

@@ -10,7 +10,6 @@ import CloseIcon from 'mdi-react/CloseIcon';
 import ArrowLeftIcon from 'mdi-react/ArrowLeftIcon';
 import { Link } from 'react-router-dom';
 import classNames from 'classnames';
-import { bool, node, string, object } from 'prop-types';
 import AppBar from '../AppBar';
 import Sidebar from '../Sidebar';
 
@@ -194,28 +193,3 @@ class Dashboard extends Component {
     );
   }
 }
-Dashboard.propTypes = {
-  /*
-   * The title to be put in the AppBar
-   */
-  title: string,
-  /*
-   * Boolean to render the sidebar
-   */
-  withSidebar: bool,
-  /*
-   * The custom header to be put in the AppBar
-   */
-  header: node,
-  /*
-   * The style to be used by the Component
-   */
-  classes: object,
-};
-
-Dashboard.defaultProps = {
-  classes: null,
-  withSidebar: false,
-  title: null,
-  header: null,
-};

src/components/DataTable/index.jsx

@@ -6,7 +6,6 @@ import TableCell from '@material-ui/core/TableCell';
 import TableHead from '@material-ui/core/TableHead';
 import TableRow from '@material-ui/core/TableRow';
 import TableSortLabel from '@material-ui/core/TableSortLabel';
-import { arrayOf, func, string, oneOf, object } from 'prop-types';
 
 @withStyles((theme) => ({
   table: {
@@ -67,44 +66,4 @@ class DataTable extends Component {
     );
   }
 }
-DataTable.propTypes = {
-  /**
-   * A function to execute for each row to render in the table.
-   * Will be passed a datum from the table data. The function
-   * should return the JSX necessary to render the given row.
-   */
-  renderRow: func,
-  /**
-   * A function to execute when a column header is clicked.
-   * Will receive a single argument which is the column name.
-   * This can be used to handle sorting.
-   */
-  onHeaderClick: func,
-  /**
-   * A header name to sort on.
-   */
-  sortByHeader: string,
-  /**
-   * The sorting direction.
-   */
-  sortDirection: oneOf(['desc', 'asc']),
-  /**
-   * A list of header names to use on the table starting from the left.
-   */
-  headers: arrayOf(string),
-  /**
-   * A list of objects to display. Each element in the list is represented
-   * by a row and each element's key-value pair represents a column.
-   */
-  items: arrayOf(object).isRequired,
-};
-
-DataTable.defaultProps = {
-  sortByHeader: null,
-  sortDirection: 'desc',
-  headers: null,
-  onHeaderClick: null,
-  renderRow: null,
-};
-
 export default DataTable;

src/components/ErrorPanel/ErrorBox.jsx

@@ -1,5 +1,4 @@
 import React from 'react';
-import { instanceOf } from 'prop-types';
 import { RedBoxError } from 'redbox-react';
 import ErrorStackParser from 'error-stack-parser';
 import { withStyles } from '@material-ui/core/styles';
@@ -53,6 +52,3 @@ class ErrorBox extends RedBoxError {
     }
   }
 }
-ErrorBox.propTypes = {
-  error: instanceOf(Error).isRequired,
-};

src/components/ErrorPanel/index.jsx

@@ -1,5 +1,4 @@
 import React, { Component } from 'react';
-import { instanceOf, oneOfType, string } from 'prop-types';
 import classNames from 'classnames';
 import Markdown from 'react-markdown';
 import { withStyles } from '@material-ui/core/styles';
@@ -80,9 +79,3 @@ class ErrorPanel extends Component {
     );
   }
 }
-ErrorPanel.propTypes = {
-  /**
-   * Error to display
-   */
-  error: oneOfType([string, instanceOf(Error)]).isRequired,
-};

src/components/ProjectCard/index.jsx

@@ -1,5 +1,4 @@
 import React, { Component } from 'react';
-import { string, shape } from 'prop-types';
 import { withStyles } from '@material-ui/core/styles';
 import { Link } from 'react-router-dom';
 import Card from '@material-ui/core/Card';
@@ -79,11 +78,3 @@ class ProjectCard extends Component {
     );
   }
 }
-ProjectCard.propTypes = {
-  project: shape({
-    name: string.isRequired,
-    summary: string,
-    fileName: string.isRequired,
-    icon: string,
-  }).isRequired,
-};

src/components/ProjectIcon/index.jsx

@@ -1,5 +1,4 @@
 import React, { Component } from 'react';
-import { string } from 'prop-types';
 import { withStyles } from '@material-ui/core/styles';
 import WebIcon from 'mdi-react/WebIcon';
 
@@ -121,7 +120,3 @@ class ProjectIcon extends Component {
     }
   }
 }
-ProjectIcon.propTypes = {
-  // The name of the icon to display, from the case statement above.
-  icon: string,
-};

src/components/ProjectIntroductionCard/index.jsx

@@ -1,6 +1,5 @@
 import React, { Component } from 'react';
 import classNames from 'classnames';
-import { string } from 'prop-types';
 import { withStyles } from '@material-ui/core/styles';
 import Card from '@material-ui/core/Card';
 import CardActions from '@material-ui/core/CardActions';
@@ -73,6 +72,3 @@ class ProjectIntroductionCard extends Component {
     );
   }
 }
-ProjectIntroductionCard.propTypes = {
-  introduction: string.isRequired,
-};

src/components/SearchBox/index.jsx

@@ -1,5 +1,4 @@
 import React, { Component } from 'react';
-import { func, string } from 'prop-types';
 import { withStyles } from '@material-ui/core/styles';
 import { fade } from '@material-ui/core/styles/colorManipulator';
 import MagnifyIcon from 'mdi-react/MagnifyIcon';
@@ -72,7 +71,3 @@ class Search extends Component {
     );
   }
 }
-Search.propTypes = {
-  value: string.isRequired,
-  onChange: func.isRequired,
-};

src/components/Sidebar/GoIcon.jsx

@@ -1,5 +1,4 @@
 import React, { PureComponent } from 'react';
-import { number, string } from 'prop-types';
 
 export default class GoIcon extends PureComponent {
   render() {
@@ -8,8 +7,8 @@ export default class GoIcon extends PureComponent {
     return (
       <svg
         className={className}
-        width={size}
-        fill={fill}
+        width={size || 24}
+        fill={fill || 'currentColor'}
         version="1.1"
         x="0px"
         y="0px"
@@ -23,13 +22,3 @@ export default class GoIcon extends PureComponent {
     );
   }
 }
-
-GoIcon.propTypes = {
-  size: number,
-  fill: string,
-};
-
-GoIcon.defaultProps = {
-  size: 24,
-  fill: 'currentColor',
-};