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

chore: replace ansi_term with nu-ansi-term #835

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

tottoto
Copy link
Contributor

@tottoto tottoto commented Aug 1, 2024

ansi_term is not maintained, and nu-ansi-term is widely used as the replacement.

@ctron
Copy link
Collaborator

ctron commented Aug 2, 2024

I was hesitant at first, but checking the dependencies, it shows that nu-ansi-term is already in the deps tree and this only drops ansi_term.

@ctron ctron merged commit 9756416 into trunk-rs:main Aug 2, 2024
57 checks passed
@tottoto tottoto deleted the replace-ansi-term-with-nu-ansi-term branch August 2, 2024 09:08
Sign up for free to 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.

2 participants