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

Debump and bump versions #687

Closed
wants to merge 6 commits into from
Closed

Debump and bump versions #687

wants to merge 6 commits into from

Conversation

koivunej
Copy link
Collaborator

CC: #668, #557

This first reverts the commit which bumped the versions to 1.0.0, then bumps the minor versions for the crates we know have in the past identified have received breaking changes.

If you go through the invidiual commits you can see how much churn a single version bump causes. Given how the interledger crate depends on all public api across crates, it does make me think if it would be best for this codebase to just have non-crates.io softer releases as the current dependency jungle is quite thick, with the re-exports the breakage will be difficult to track.

There's no hurry on my part, but this puts in code what I've been thinking recently and commenting on the issues.

@koivunej koivunej marked this pull request as ready for review March 10, 2021 10:30
This reverts commit 3f1a8de. This
restores rather large spread of version numbers across the many crates.

Signed-off-by: Joonas Koivunen <[email protected]>
for public api changes.

Signed-off-by: Joonas Koivunen <[email protected]>
for public api changes

Signed-off-by: Joonas Koivunen <[email protected]>
for public api changes.

Signed-off-by: Joonas Koivunen <[email protected]>
it re-exports all of the api so any api breakage must be reflected on
it's version.

Signed-off-by: Joonas Koivunen <[email protected]>
for the past work, there have been no API breaking changes however.

Signed-off-by: Joonas Koivunen <[email protected]>
@koivunej koivunej closed this by deleting the head repository May 5, 2023
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.

1 participant