-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: remove default props #6461
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6461 +/- ##
========================================
Coverage 89.09% 89.09%
========================================
Files 730 730
Lines 31827 31840 +13
Branches 5771 6121 +350
========================================
+ Hits 28355 28367 +12
+ Misses 3426 3278 -148
- Partials 46 195 +149
... and 63 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Description
Removes instances of defaultProps as this is slated to be removed from React in a future version: facebook/react#29233. Components, where a default prop was provided, had their associated props set to optional.
Test plan
Related issues
N/A
Backwards compatibility
Yes
Network scalability
N/A