feat(render,cli)!: move minify, pretty, strip into render. combine head tags - Githubissues

shellscape / jsx-email

Build emails with a delightful DX

https://jsx.email

MIT License

902 stars 28 forks source link

feat(render,cli)!: move minify, pretty, strip into render. combine head tags #59

Closed shellscape closed 8 months ago

shellscape commented 8 months ago

Component / Package Name: `render`

This PR contains:

[x] bugfix
[ ] feature
[ ] refactor
[ ] documentation
[ ] other

Are tests included?

[x] yes (bugfixes and features will not be merged without tests)
[ ] no

Breaking Changes?

[x] yes (breaking changes will not be merged unless absolutely necessary)
[ ] no

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

BREAKING CHANGES

This PR does a few things:

removes renderAsync and makes the default render method async. There is no longer a sync render method.
moves the minify, pretty, and strip functions to the renderer and out of the cli
combines the contents of multiple <head> elements into one (the first in the document). this is primarily for an upcoming major release for the Tailwind component