-
Notifications
You must be signed in to change notification settings - Fork 193
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
Add minimum supported rustc version #143
Conversation
Why did you specifically pick 1.18? Just curious |
It was a bit of an arbitrary decision, it's a release that I think most package managers support. |
We need to check with some package managers before going for it, just to be sure |
1.18 is old enough for most crate owners to release a new version. I have seen a lot of crates keep 1.18 as their min dependency which is fine imo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fair point. People using older versions should ideally update by now
bors: r+ |
143: Add minimum supported rustc version r=Dylan-DPC a=KodrAus Specifically test against `rustc 1.18.0` so we know when we regress the minimum supported version. Personally, I'm ok with pushing the minimum version as a non-breaking change, but it's always good to know and communicate in the release notes.
143: Add minimum supported rustc version r=Dylan-DPC a=KodrAus Specifically test against `rustc 1.18.0` so we know when we regress the minimum supported version. Personally, I'm ok with pushing the minimum version as a non-breaking change, but it's always good to know and communicate in the release notes.
bors: r+ |
143: Add minimum supported rustc version r=Dylan-DPC a=KodrAus Specifically test against `rustc 1.18.0` so we know when we regress the minimum supported version. Personally, I'm ok with pushing the minimum version as a non-breaking change, but it's always good to know and communicate in the release notes.
bors: r+ |
Not awaiting review |
bors r- |
Canceled |
bors r+ |
143: Add minimum supported rustc version r=kinggoesgaming a=KodrAus Specifically test against `rustc 1.18.0` so we know when we regress the minimum supported version. Personally, I'm ok with pushing the minimum version as a non-breaking change, but it's always good to know and communicate in the release notes.
Specifically test against
rustc 1.18.0
so we know when we regress the minimum supported version. Personally, I'm ok with pushing the minimum version as a non-breaking change, but it's always good to know and communicate in the release notes.