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

Use std::io::IsTerminal (stabilized in 1.70) over crossterm implementation. #7605

Closed
wants to merge 2 commits into from

Conversation

gibbz00
Copy link
Contributor

@gibbz00 gibbz00 commented Jul 11, 2023

@pascalkuthe
Copy link
Member

according to our Msrv policy we never bump our MSRV beyond that required by firefox stable which is currently at 1.66 (our msrv is 1.65 right now) and likely won't reach 1.70 this year. It's not worth breaking that policy for something that makes no functional difference

@gibbz00
Copy link
Contributor Author

gibbz00 commented Jul 11, 2023

Yup, just leaving it open for now 👍

@pascalkuthe
Copy link
Member

while our msrv is new enough now this has conflicts and will only be useful once this removed from crossterm aswell (at which point we will have to update anyway). So I am closing this out. Thanks for contributing

@pascalkuthe pascalkuthe closed this Apr 8, 2024
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