Note: Would make sure that no distinguishable changes can be seen in the demo. None to my eyes, but can't hurt to double check.
Key takeaways:
Remove unnecessary use of React.forwardRef
Change @jsx-email/components imports to @jsx-email/all
Reduce React import to typed imports where possible (although I don't think this will provide a library benefit seeing as react is only listed as a peer dependency in each package)
This PR contains:
[ ] bugfix
[ ] feature
[x] refactor
[ ] documentation
[ ] other
Are tests included?
[ ] yes (bugfixes and features will not be merged without tests)
[x] no
Breaking Changes?
[ ] yes (breaking changes will not be merged unless absolutely necessary)
[x] no
If yes, please include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.
lordelogos
commented
11 months ago
Component / Package Name: N/A
Note:Would make sure that no distinguishable changes can be seen in the demo. None to my eyes, but can't hurt to double check.Key takeaways:
React.forwardRef@jsx-email/componentsimports to@jsx-email/allreactis only listed as a peer dependency in each package)This PR contains:
Are tests included?
Breaking Changes?
If yes, please include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.
List any relevant issue numbers:
Description