Refactor use of futures crate

tower-rs / tower-http

HTTP specific Tower utilities.

680 stars 159 forks source link

Refactor use of futures crate #434

Closed tottoto closed 10 months ago

tottoto commented 10 months ago

Motivation

Refactoring.

Solution

Replaces futures_{util,core} with std's APIs when possible.
Uses futures_core's API via futures_util as already used at the other places.
Replaces futures with futures_util.
Makes futures_util dependency optional.

jplatte commented 10 months ago

Hm, I'm unsure whether we should really remove the futures crate usage from examples. Could you split that change into a separate PR or open an issue for discussion? Then I'd be happy to merge the remaining changes.

tottoto commented 10 months ago

Fixed.

getong commented 10 months ago

http 1.0 is released, how about upgrading to http 1.0?

jplatte commented 10 months ago

http 1.0 is released, how about upgrading to http 1.0?

That's what #348 does. Why are you asking on this PR?

getong commented 10 months ago

sorry.

jplatte commented 10 months ago

@getong no worries, I was just a bit confused by it.