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

Release axum 0.8.2 and related crates #3188

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Release axum 0.8.2 and related crates #3188

merged 1 commit into from
Jan 21, 2025

Conversation

jplatte
Copy link
Member

@jplatte jplatte commented Jan 21, 2025

We've had a couple of useful changes, release them.

@jplatte jplatte requested review from yanns, Turbo87 and mladedav January 21, 2025 13:01
@jplatte jplatte merged commit 5966849 into main Jan 21, 2025
18 checks passed
@jplatte jplatte deleted the jplatte/axum-v0.8.2 branch January 21, 2025 13:12
@evanrichter
Copy link

so you are aware, v0.8.2 is not on crates.io, despite the github release and tag

@jplatte
Copy link
Member Author

jplatte commented Feb 19, 2025

It is on crates.io, but in yanked state. I'm currently very busy with real life so haven't put time into 0.8.3.

If you or anybody else wants to move things along, you can read up on why the release was yanked in the closed issue about breaking changes. Then as a next step it would be good to open a discussion proposing a way forward regarding the changes that lead to breakage (a simple solution might be to just revert the const fn change that caused issues, pubpish new releases and revisit that change afterwards).

@evanrichter
Copy link

ah sorry, I didn't see anything about the yank. maybe it could be added as a line in the github release.

I don't mean to add pressure to release 0.8.3, it just seemed like something was missed

@mladedav
Copy link
Collaborator

I've added a line on top of that release to tell future visitor that it has been yanked. Thanks for the suggestion.

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