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

v10.1.0 increases the MSRV to 1.62 #671

Closed
daxpedda opened this issue Jan 17, 2023 · 9 comments
Closed

v10.1.0 increases the MSRV to 1.62 #671

daxpedda opened this issue Jan 17, 2023 · 9 comments

Comments

@daxpedda
Copy link

#662 uses derive(Default) on enums, which is only supported since 1.62.
Personally I don't have a stake in supporting old MSRV's, but it broke CI's on projects I'm working on.

Cc @kevinlewi, I assume you have a stake in this.

@gwenn
Copy link
Collaborator

gwenn commented Jan 17, 2023

@daxpedda
Copy link
Author

Maybe there is a misunderstand there.
The crate rustyline cannot be used with Rust versions lower then 1.62 because it uses the #[derive(Default)] on enums feature.

Yes, the Clippy configuration you posted would be useful to add to rustyline to prevent this problem in the future.
I can make a PR.

@gwenn
Copy link
Collaborator

gwenn commented Jan 18, 2023

But currently there is no MSRV defined for rustyline.
We only define edition = "2018".
So I guess MSRV should be <= 1.55.0 (edition "2021 = version 1.56.0).

@gwenn
Copy link
Collaborator

gwenn commented Jan 18, 2023

Except that env_logger version 0.10.0 as an MSRV = 1.60.0.
So 5abc48d should also be rollbacked.

@gwenn
Copy link
Collaborator

gwenn commented Jan 18, 2023

@daxpedda
Copy link
Author

I guess this is a request to add an MSRV to rustyline then?
Would it be possible to make a new release with the MSRV branch?

@daxpedda
Copy link
Author

Thank you for the prompt handling of all this ❤️.

@gwenn
Copy link
Collaborator

gwenn commented Jan 19, 2023

4 days since 10.1.0 release...
And sorry for all the troubles.

@daxpedda daxpedda mentioned this issue Jan 19, 2023
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 a pull request may close this issue.

2 participants