fix(tailwind): incorrect variable reference in processHead()

phpnode commented 11 months ago

Component / Package Name: @jsx-email/tailwind

This PR contains:

[x] bugfix
[ ] feature
[ ] 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.

List any relevant issue numbers:

Description

processHead() was previously returning child instead of processedChild, leading to duplicated output when using <Tailwind />

shellscape commented 11 months ago

awesome, thanks. will follow up when CI is done running

shellscape commented 11 months ago

OK as I thought, this was probably going to break the tests we have in place. Please run moon tailwind:test -- -u to update the test snapshots, and commit the updates.

shellscape commented 11 months ago

Here's a link to the diff for this change. Looks legit https://www.diffchecker.com/lkjdtOWr/

shellscape commented 11 months ago

Updated your branch with the test updates so we could get this fix out. Hope you don't mind :)

shellscape / jsx-email

fix(tailwind): incorrect variable reference in processHead() #33

Component / Package Name: @jsx-email/tailwind

Description