From 3f920cc68cbc6cb12fb1b91113ef0baa6e91d8aa Mon Sep 17 00:00:00 2001 From: Nathan Bierema Date: Mon, 2 Jan 2023 21:46:43 -0500 Subject: [PATCH] Remove proptypes --- packages/react-json-tree/package.json | 2 -- packages/react-json-tree/src/ItemRange.tsx | 9 -------- .../react-json-tree/src/JSONArrayNode.tsx | 5 ----- packages/react-json-tree/src/JSONArrow.tsx | 15 +------------ .../react-json-tree/src/JSONNestedNode.tsx | 21 ------------------- packages/react-json-tree/src/JSONNode.tsx | 13 ------------ .../react-json-tree/src/JSONObjectNode.tsx | 6 ------ .../react-json-tree/src/JSONValueNode.tsx | 13 ------------ pnpm-lock.yaml | 4 ---- 9 files changed, 1 insertion(+), 87 deletions(-) diff --git a/packages/react-json-tree/package.json b/packages/react-json-tree/package.json index 30708af4..f985868a 100644 --- a/packages/react-json-tree/package.json +++ b/packages/react-json-tree/package.json @@ -47,8 +47,6 @@ "dependencies": { "@babel/runtime": "^7.20.6", "@types/lodash": "^4.14.191", - "@types/prop-types": "^15.7.5", - "prop-types": "^15.8.1", "react-base16-styling": "^0.9.1" }, "devDependencies": { diff --git a/packages/react-json-tree/src/ItemRange.tsx b/packages/react-json-tree/src/ItemRange.tsx index 7510ae52..7eb1cb03 100644 --- a/packages/react-json-tree/src/ItemRange.tsx +++ b/packages/react-json-tree/src/ItemRange.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import JSONArrow from './JSONArrow'; import type { CircularCache, CommonInternalProps } from './types'; @@ -17,14 +16,6 @@ interface State { } export default class ItemRange extends React.Component { - static propTypes = { - styling: PropTypes.func.isRequired, - from: PropTypes.number.isRequired, - to: PropTypes.number.isRequired, - renderChildNodes: PropTypes.func.isRequired, - nodeType: PropTypes.string.isRequired, - }; - constructor(props: Props) { super(props); this.state = { expanded: false }; diff --git a/packages/react-json-tree/src/JSONArrayNode.tsx b/packages/react-json-tree/src/JSONArrayNode.tsx index 778b5f5f..eadf3a6c 100644 --- a/packages/react-json-tree/src/JSONArrayNode.tsx +++ b/packages/react-json-tree/src/JSONArrayNode.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import JSONNestedNode from './JSONNestedNode'; import type { CommonInternalProps } from './types'; @@ -28,8 +27,4 @@ const JSONArrayNode: React.FunctionComponent = ({ data, ...props }) => ( /> ); -JSONArrayNode.propTypes = { - data: PropTypes.array, -}; - export default JSONArrayNode; diff --git a/packages/react-json-tree/src/JSONArrow.tsx b/packages/react-json-tree/src/JSONArrow.tsx index c2bf1c26..8caa2a9a 100644 --- a/packages/react-json-tree/src/JSONArrow.tsx +++ b/packages/react-json-tree/src/JSONArrow.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import type { StylingFunction } from 'react-base16-styling'; interface Props { @@ -12,7 +11,7 @@ interface Props { const JSONArrow: React.FunctionComponent = ({ styling, - arrowStyle, + arrowStyle = 'single', expanded, nodeType, onClick, @@ -27,16 +26,4 @@ const JSONArrow: React.FunctionComponent = ({ ); -JSONArrow.propTypes = { - styling: PropTypes.func.isRequired, - arrowStyle: PropTypes.oneOf(['single', 'double']), - expanded: PropTypes.bool.isRequired, - nodeType: PropTypes.string.isRequired, - onClick: PropTypes.func.isRequired, -}; - -JSONArrow.defaultProps = { - arrowStyle: 'single', -}; - export default JSONArrow; diff --git a/packages/react-json-tree/src/JSONNestedNode.tsx b/packages/react-json-tree/src/JSONNestedNode.tsx index fdd172f1..38a67373 100644 --- a/packages/react-json-tree/src/JSONNestedNode.tsx +++ b/packages/react-json-tree/src/JSONNestedNode.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import JSONArrow from './JSONArrow'; import getCollectionEntries from './getCollectionEntries'; import JSONNode from './JSONNode'; @@ -111,26 +110,6 @@ function getStateFromProps(props: Props) { } export default class JSONNestedNode extends React.Component { - static propTypes = { - getItemString: PropTypes.func.isRequired, - nodeTypeIndicator: PropTypes.any, - nodeType: PropTypes.string.isRequired, - data: PropTypes.any, - hideRoot: PropTypes.bool.isRequired, - createItemString: PropTypes.func.isRequired, - styling: PropTypes.func.isRequired, - collectionLimit: PropTypes.number, - keyPath: PropTypes.arrayOf( - PropTypes.oneOfType([PropTypes.string, PropTypes.number]) - ).isRequired, - labelRenderer: PropTypes.func.isRequired, - shouldExpandNode: PropTypes.func, - level: PropTypes.number.isRequired, - sortObjectKeys: PropTypes.oneOfType([PropTypes.func, PropTypes.bool]), - isCircular: PropTypes.bool, - expandable: PropTypes.bool, - }; - static defaultProps = { data: [], circularCache: [], diff --git a/packages/react-json-tree/src/JSONNode.tsx b/packages/react-json-tree/src/JSONNode.tsx index c31f337b..92c6d8a1 100644 --- a/packages/react-json-tree/src/JSONNode.tsx +++ b/packages/react-json-tree/src/JSONNode.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import objType from './objType'; import JSONObjectNode from './JSONObjectNode'; import JSONArrayNode from './JSONArrayNode'; @@ -103,16 +102,4 @@ const JSONNode: React.FunctionComponent = ({ } }; -JSONNode.propTypes = { - getItemString: PropTypes.func.isRequired, - keyPath: PropTypes.arrayOf( - PropTypes.oneOfType([PropTypes.string, PropTypes.number]).isRequired - ).isRequired, - labelRenderer: PropTypes.func.isRequired, - styling: PropTypes.func.isRequired, - value: PropTypes.any, - valueRenderer: PropTypes.func.isRequired, - isCustomNode: PropTypes.func.isRequired, -}; - export default JSONNode; diff --git a/packages/react-json-tree/src/JSONObjectNode.tsx b/packages/react-json-tree/src/JSONObjectNode.tsx index 6cdf4711..7d1e10a1 100644 --- a/packages/react-json-tree/src/JSONObjectNode.tsx +++ b/packages/react-json-tree/src/JSONObjectNode.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import JSONNestedNode from './JSONNestedNode'; import type { CommonInternalProps } from './types'; @@ -27,9 +26,4 @@ const JSONObjectNode: React.FunctionComponent = ({ data, ...props }) => ( /> ); -JSONObjectNode.propTypes = { - data: PropTypes.object, - nodeType: PropTypes.string.isRequired, -}; - export default JSONObjectNode; diff --git a/packages/react-json-tree/src/JSONValueNode.tsx b/packages/react-json-tree/src/JSONValueNode.tsx index 99078848..c20910de 100644 --- a/packages/react-json-tree/src/JSONValueNode.tsx +++ b/packages/react-json-tree/src/JSONValueNode.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import type { GetItemString, Key, @@ -44,16 +43,4 @@ const JSONValueNode: React.FunctionComponent = ({ ); -JSONValueNode.propTypes = { - nodeType: PropTypes.string.isRequired, - styling: PropTypes.func.isRequired, - labelRenderer: PropTypes.func.isRequired, - keyPath: PropTypes.arrayOf( - PropTypes.oneOfType([PropTypes.string, PropTypes.number]).isRequired - ).isRequired, - valueRenderer: PropTypes.func.isRequired, - value: PropTypes.any, - valueGetter: PropTypes.func, -}; - export default JSONValueNode; diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index f0cfa35b..f029d298 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -536,7 +536,6 @@ importers: '@types/jest': ^29.2.4 '@types/lodash': ^4.14.191 '@types/node': ^18.11.17 - '@types/prop-types': ^15.7.5 '@types/react': ^18.0.26 '@types/react-test-renderer': ^18.0.0 '@typescript-eslint/eslint-plugin': ^5.47.0 @@ -547,7 +546,6 @@ importers: eslint-plugin-react: ^7.31.11 eslint-plugin-react-hooks: ^4.6.0 jest: ^29.3.1 - prop-types: ^15.8.1 react: ^18.2.0 react-base16-styling: ^0.9.1 react-test-renderer: ^18.2.0 @@ -560,8 +558,6 @@ importers: dependencies: '@babel/runtime': 7.20.6 '@types/lodash': 4.14.191 - '@types/prop-types': 15.7.5 - prop-types: 15.8.1 react-base16-styling: link:../react-base16-styling devDependencies: '@babel/cli': 7.19.3_@babel+core@7.20.5