Skip to content
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

refactor: replace strip-ansi with native module #2521

Merged

Conversation

stianjensen
Copy link
Contributor

Inspired by eslint/eslint#18982

This API is built-in since node 16.11. react-native-cli already requires node 18 as of last year.

Copy link
Collaborator

@szymonrybczak szymonrybczak left a comment

Choose a reason for hiding this comment

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

Thanks!

@stianjensen
Copy link
Contributor Author

Not sure if this windows snapshot diff could be related to this PR or not?

@szymonrybczak
Copy link
Collaborator

@stianjensen not really, failure on E2E Windows pipeline was caused by other change that landed on main branch. This PR should be good to merge 👍

Copy link

github-actions bot commented Feb 4, 2025

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.

@github-actions github-actions bot added the stale label Feb 4, 2025
Inspired by eslint/eslint#18982

This API is built-in since node 16.11. react-native-cli already requires
node 18 as of last year.
@szymonrybczak szymonrybczak merged commit 0dcdcc6 into react-native-community:main Feb 4, 2025
8 checks passed
@stianjensen stianjensen deleted the remove-strip-ansi branch February 4, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants