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

Bump versions of semver-broken crates to {N+1}-dev #5769

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

robertbastian
Copy link
Member

@robertbastian robertbastian commented Nov 4, 2024

I think we should in the future do this as soon as we make a semver-breaking change anywhere, as it's much harder to do this after the fact. We can use the -dev tag to avoid bumping a crate twice, and it's easy to remove the tag before a release (or replace with beta).

@robertbastian robertbastian changed the title Bump versions to beta Bump versions to 2.0.0-beta1 Nov 4, 2024
@robertbastian robertbastian mentioned this pull request Nov 4, 2024
@robertbastian robertbastian force-pushed the beta branch 2 times, most recently from 9bb4375 to 80f6078 Compare November 4, 2024 12:29
robertbastian added a commit that referenced this pull request Nov 4, 2024
Quick fix until #5769 lands.
@robertbastian robertbastian changed the title Bump versions to 2.0.0-beta1 Bump versions of semver-broken crates to {N+1}-dev Nov 4, 2024
Copy link
Member

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clever

@Manishearth Manishearth merged commit 7a2aa53 into unicode-org:main Nov 4, 2024
28 checks passed
@Manishearth
Copy link
Member

Merging to fix main

@robertbastian robertbastian deleted the beta branch November 4, 2024 17:49
@sffc
Copy link
Member

sffc commented Nov 4, 2024

Strongly in support of this. It should be a release task to add "-dev" on the next version after we publish the release.

@robertbastian
Copy link
Member Author

It should only be added once there's a change that requires a version bump.

@Manishearth
Copy link
Member

I think for utils crates we should not add -dev until we have to. For ICU4X crates, where we have ~ deps anyway, I don't thin there's harm in preemptively adding it, we know what the next version will be (and patches will be released off-main anyway).

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.

4 participants