Skip to content

Conversation

@kym6464
Copy link
Contributor

@kym6464 kym6464 commented Jun 23, 2023

Default props is deprecated for functional components facebook/react#25699 and gives this error the console:

Warning: App: Support for defaultProps will be removed from function components in a future major release. Use JavaScript default parameters instead.

This PR implements the recommended solution. I ran all the unit test, integration tests, and all of the examples. Everything seems to be working as before!

"scripts": {
"prepublishOnly": "npm run build",
"clean": "rimraf dist",
"clean": "rimraf dist && rimraf node_modules/.cache",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rollup was not seeing my changes to types.ts and I had to manually delete this directory to fix it. Figured it'd be good to automate

@benhowell benhowell merged commit 2f49ba9 into benhowell:master Jun 24, 2023
@benhowell
Copy link
Owner

Thanks @kym6464, very much appreciated. Code looks good and the auto removal of .cache is a nice bonus!
Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants