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

[feature] remove the versions (crates are unpublished) #785

Merged
merged 1 commit into from
Jun 7, 2021

Conversation

woshilapin
Copy link
Contributor

@woshilapin woshilapin commented Jun 4, 2021

Having both path and version is useful only when the crate is published. From the documentation, path is always used, except for publish where `version is used. Since we don't release these crates, versions are not needed.

NOTE: we also want to avoid a situation where we would publish a crate gtfs2ntfs:0.36.1 with a dependency transit_model:0.36.0 which would be possible with a dependency specified like version = "0.36". Therefore, the rational here is that since we don't publish these crates at all, removing the version is a way to push back the problem to the day we will decide to do it (or not !).

@woshilapin woshilapin requested a review from pbougue June 4, 2021 15:24
@woshilapin woshilapin merged commit 5564b51 into master Jun 7, 2021
@woshilapin woshilapin deleted the remove-version branch June 7, 2021 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants