Documentation: HttpConnector requires the "tcp" feature

[tux3]

The docs.rs documentation for HttpConnector with hyper 0.14.1 says:

This is supported on crate feature client and (crate features http1 or http2) only.

As I understand it this seems based on of the following in hyper's lib.rs:

cfg_feature! {
    #![all(feature = "client", any(feature = "http1", feature = "http2"))]

    pub mod client;
    pub use crate::client::Client;
}

But it seems the HttpConnector struct also depends on the tcp feature.

#[cfg(feature = "tcp")]
pub use self::connect::HttpConnector;

I'm not sure how much control you have over this, but would it be possible to fix the doc to show the extra required feature? =)

[seanmonstar]

True, this could be fixed by changing it to use the cfg_feature macro instead, like:

cfg_feature {
    #![feature = "tcp"]
    pub use self::connect::HttpConnector;
}

[SkamDart]

Looks like I need to fix the commit message but what is the right way to test this?
I can't quite figure out the incantation that docs.rs uses to generate docs...

[seanmonstar]

Fixed in #2377