Skip to content

chore: migrate from chalk to picocolors - #2631

Merged
thymikee merged 17 commits into
react-native-community:mainfrom
Rexogamer:main
Dec 16, 2025
Merged

chore: migrate from chalk to picocolors#2631
thymikee merged 17 commits into
react-native-community:mainfrom
Rexogamer:main

Conversation

@Rexogamer

Copy link
Copy Markdown
Contributor

Summary

picocolors is much smaller and slightly faster than chalk. this PR migrates the various CLI packages to picocolors.

Test Plan

yarn test - worked fine with one minor change (see review comments). the output should be identical.

Checklist

  • Documentation is up to date.
  • Follows commit message convention described in CONTRIBUTING.md.

Comment threadpackages/cli-config/src/__tests__/__snapshots__/index-test.ts.snap Outdated
Comment threadpackage.json Outdated

@szymonrybczakszymonrybczak left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you! 👌

@Rexogamer

Copy link
Copy Markdown
ContributorAuthor

rebased; should be mergeable

@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 3 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 7 days.

@Rexogamer

Copy link
Copy Markdown
ContributorAuthor

will rebase this soon

@Rexogamer

Copy link
Copy Markdown
ContributorAuthor

okay this should be ready

@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 3 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 7 days.

@Rexogamer

Copy link
Copy Markdown
ContributorAuthor

(I can't currently see if there are any conflicts but assuming any are fixed) is this likely to be merged?

@thymikeethymikee removed the stale label Dec 3, 2025
@thymikee

Copy link
Copy Markdown
Member

yup, we should be able to merge it once conflicts are resolved :)

@stianjensen

stianjensen commented Dec 3, 2025

Copy link
Copy Markdown
Contributor
Screenshot 2025-12-03 at 17 52 17

Really feels like these two contradict eachother 🤔

@Rexogamer

Copy link
Copy Markdown
ContributorAuthor

this should be ready now :3

@thymikee
thymikee merged commit d20b58c into react-native-community:mainDec 16, 2025
4 of 8 checks passed
@thymikee

Copy link
Copy Markdown
Member

Thank you @Rexogamer!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Rexogamer@thymikee@stianjensen@szymonrybczak