Skip to content

Conversation

tottoto
Copy link
Collaborator

@tottoto tottoto commented Oct 2, 2025

Motivation

Since axum-extra's typed-routing feature requires routing feature, both are needed to be enabled to be checked.

Solution

Adds an check of axum-extra with enabling both of routing and typed-routing features.

@tottoto tottoto force-pushed the check-axum-extra-typed-routing branch from f9f1dc8 to 63513eb Compare October 11, 2025 06:43
@jplatte
Copy link
Member

jplatte commented Oct 11, 2025

I see, this is due to #3485. But I don't think this is the right solution. Shouldn't we just make the typed-routing feature pull in the routing feature, instead of being a no-op on its own?

@tottoto tottoto force-pushed the check-axum-extra-typed-routing branch from 63513eb to bb2b5bc Compare October 11, 2025 08:02
@tottoto tottoto changed the title ci: Check axum-extra typed-routing feature axum-extra: Make typed-routing feature enable routing feature Oct 11, 2025
@tottoto
Copy link
Collaborator Author

tottoto commented Oct 11, 2025

Yes, that is also possible. The original change was intended to fix the current check. Updated.

@jplatte jplatte merged commit dcbcf5c into tokio-rs:main Oct 11, 2025
18 checks passed
@tottoto tottoto deleted the check-axum-extra-typed-routing branch October 11, 2025 09:30
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