Skip to content

Cleanup (mostly axum related)#170

Merged
jplatte merged 8 commits intomasterfrom
jplatte/axum0.8
Jan 9, 2025
Merged

Cleanup (mostly axum related)#170
jplatte merged 8 commits intomasterfrom
jplatte/axum0.8

Conversation

@jplatte
Copy link
Collaborator

@jplatte jplatte commented Jan 8, 2025

This should now work without #171. See individual commit messages for details.

I'd like to publish aide v0.14.0 once this is merged, unless you're aware of other things that should happen before that release.

They have not upgraded to axum 0.8 yet.
We can alwas re-introduce them later.
It's not just required for tests.
- Don't use the full path to the trait, it's in the prelude
- Put it right after Send in bounds
@jplatte jplatte changed the title Upgrade dependencies Clean up (mostly axum related) Jan 9, 2025
@jplatte jplatte force-pushed the jplatte/axum0.8 branch 2 times, most recently from cee0826 to ab5e5f1 Compare January 9, 2025 09:29
@jplatte jplatte changed the title Clean up (mostly axum related) Cleanup (mostly axum related) Jan 9, 2025
@jplatte jplatte marked this pull request as draft January 9, 2025 12:01
- Move things behind features that weren't
- Replace non-additive axum-wasm feature with additive axum-tokio feature
@jplatte jplatte marked this pull request as ready for review January 9, 2025 12:06
Copy link
Collaborator

@JakkuSakura JakkuSakura left a comment

Choose a reason for hiding this comment

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

LGTM

@jplatte jplatte merged commit 2fa7282 into master Jan 9, 2025
1 check passed
@jplatte jplatte deleted the jplatte/axum0.8 branch January 9, 2025 16:59
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.

2 participants