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

Switch to std::io::IsTerminal for v3 #144

Closed
kurtlawrence opened this issue Jul 4, 2023 · 4 comments
Closed

Switch to std::io::IsTerminal for v3 #144

kurtlawrence opened this issue Jul 4, 2023 · 4 comments
Labels
breaking Will need a major version bump

Comments

@kurtlawrence
Copy link
Collaborator

For the v3 release, switch to using std::io::IsTerminal.

@kurtlawrence kurtlawrence added the breaking Will need a major version bump label Jul 4, 2023
@juntyr
Copy link

juntyr commented Jul 4, 2023

+1 from me for removing the atty dependency

@Oakchris1955
Copy link
Contributor

I opened a PR (#149) which uses the built-in trait instead of the external crate. I hope it get approved by the maintainers

@hwittenborn
Copy link
Member

Thanks for the PR @Oakchris1955! I'll get that merged in once the v3 branch gets created - there's a few other PRs that need to get merged, and then I'll get that looked at.

@Oakchris1955
Copy link
Contributor

Since this was basically resolved with 2.1, shouldn't this issue be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Will need a major version bump
Projects
None yet
Development

No branches or pull requests

5 participants