-
Notifications
You must be signed in to change notification settings - Fork 684
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
Let's get rid of the -pre suffix. #468
Comments
I agree that dropping the pre should be fine within the context of the Rust/Cargo. If user's are using a pre-release version they are pointing to git and Cargo.lock will be dealing with things appropriately. |
I'd actually propose closing this and instead using |
@Susurrus Well that |
Actually I'm confused… What do you mean by |
Sorry, I meant to indicate We should clarify this in |
I wouldn't say "without issues", given there has been at least 2 issues about the release procedure being weird (#363 and this one), and again this is the only crate I know of which does that anyway. |
I agree that we can remove this step so I filed #860. |
As @nox stated in #363, no one else does it, and it apparently might cause, problems I don't fully understand, yet. Since, I don't see an advantage in having the pre in the repository all the time, I don't mind.
In addition, I would like to fix the next version to currently 0.(prev + 1).0 and not 0.prev.1, if we ever want to release a hotfix, we can do so in a separate branch with a 0.current.1 version.
I'll prepare an RFC, when I have time.
The text was updated successfully, but these errors were encountered: