-
Notifications
You must be signed in to change notification settings - Fork 828
feat(react-email, preview-server): use styleText
from node:util
instead of chalk
#2430
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
Conversation
@yoshi-taka is attempting to deploy a commit to the resend Team on Vercel. A member of the Team first needs to authorize it. |
🦋 Changeset detectedLatest commit: 9f2f637 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@react-email/button
@react-email/code-block
@react-email/code-inline
@react-email/column
@react-email/components
@react-email/container
create-email
@react-email/head
@react-email/heading
@react-email/hr
@react-email/img
@react-email/link
@react-email/preview
@react-email/preview-server
react-email
@react-email/render
@react-email/section
@react-email/text
commit: |
5cf4059
to
89a1129
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 3 files
really like this, but I don't think that's the only place we use chalk across the entire repo |
styleText
from node:util
instead of chalk
e8d27f3
to
3a97bb3
Compare
3a97bb3
to
9f2f637
Compare
Thank you so much for this! Just released it under react-email@4.2.9 so we can avoid our users getting into security issues |
good timing of removing the package :) |
This has broken support for Bun, until Bun updates their |
https://github.com/es-tooling/module-replacements/blob/main/docs/modules/chalk.md
Chalk is no longer needed since Node.js provides sufficient native support.
This change simplifies the codebase and makes installation faster.
Summary by cubic
Replaced chalk with Node’s native util.styleText in the preview dev server, removing the chalk dependency. Keeps the same colored output while simplifying setup and speeding up installs.