-
Notifications
You must be signed in to change notification settings - Fork 290
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
Update dependencies and bump MSRV to v1.70 #912
Conversation
@emilio What do you think - would bumping the MSRV to 1.70 be acceptable now, given that it is half a year old? |
This also removes the dependency on the unmaintained crate atty. Closes mozilla#828
Disable default features (logging, async), since we don't need them right now.
de2e584
to
dd88ef0
Compare
Thanks for picking this up, especially the upgrade from clap 3 to 4. Would be awesome to have a release with this :) |
Indeed, this would be super appreciated! For those of us who try to resolve vulnerability alerts on all our GitHub repositories, we're unfortunately stuck until |
@emilio Is there anything blocking a release with this in it? At this point, my use of If there's something I can do to help, I'd be glad to. |
Nothing particularly blocking other than the fact that I need to go through all the changes and prepare a changelog, which takes time I don't have right now... Any help with that is really appreciated, it should be a matter of making a PR to the |
Rust 1.70 was released on 2023-06-01, so it is half a year old by now. It is also the minimum version I got the updated dependencies to compile on.
Based on #870.
Closes #880
Closes #899