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

Encourage updating Rust #495

Closed
wants to merge 1 commit into from

Conversation

interfect
Copy link

This should help stop people from making mistakes like bevyengine/bevy#6777.

Novice Rust programmers might see "latest stable release" and think that that means "what I installed last month when I started learning Rust". However, that's not likely to be the case.

Rust makes stable releases often enough that anyone who doesn't update it every couple weeks or so actually needs to update Rust in order to use Bevy.

This should make that clearer.

This should help stop people from making mistakes like
bevyengine/bevy#6777.

Novice Rust programmers might see "latest stable release" and think that that
means "what I installed last month when I started learning Rust". However,
that's not likely to be the case.

Rust makes stable releases often enough that anyone who doesn't update it every
couple weeks or so actually needs to update Rust in order to use Bevy.

This should make that clearer.
@alice-i-cecile
Copy link
Member

Hmm. I think that this is unlikely to help users in practice: this will be read once, and then forgotten. I do think the MSRV approach discussed in that issue is much more direct.

@alice-i-cecile alice-i-cecile added A-Book C-Content X-Controversial There is active debate or serious implications around merging this PR labels Nov 27, 2022
@hymm
Copy link
Contributor

hymm commented Jan 12, 2023

should this be closed now that bevyengine/bevy#6852 has been merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Book C-Content X-Controversial There is active debate or serious implications around merging this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants