[material-ui][types] Make `OverridableComponent` component prop optional

mui / material-ui

Material UI: Ready-to-use foundational React components, free forever. It includes Material UI, which implements Google's Material Design.

https://mui.com/material-ui/

MIT License

91.86k stars 31.57k forks source link

[material-ui][types] Make `OverridableComponent` component prop optional #42097

Open sydneyjodon-wk opened 2 weeks ago

sydneyjodon-wk commented 2 weeks ago

[x] I have followed (at least) the PR section of the contributing guide.

Hello!

I'm not sure if there's a reason that the component prop in OverridableComponent is set to required, but it's coming through as required for us for a lot of components. I'm wondering if it would be okay to make it optional here instead?

Thank you!

mui-bot commented 2 weeks ago

Netlify deploy preview

https://deploy-preview-42097--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad) No bundle size changes

Generated by :no_entry_sign: dangerJS against 4d823aa7c62e08f2a0c08e858a27373cf25e5b97

sydneyjodon-wk commented 2 weeks ago

Based on the generation changes, it seems like there might have been a reason the prop is required?