-
Notifications
You must be signed in to change notification settings - Fork 211
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
Decide on MSRV #788
Comments
Is there any reason not to just use |
Also fine for me. Don't have a strong opinion on this other than having to pin a (transitive) dependency because of the MSRV is not a good thing |
I don't think we should blindly upgrade to |
#779 additionally requires If we need to bump MSRV we should make sure it's at least |
The only thing left to do to close this issue is bump to MSRV 1.67 and unpin the |
See #787
I like the
CURRENT_VERSION - 1
idea and with 7b45dca that should be easy to implementThe text was updated successfully, but these errors were encountered: