Deprecate all intermediate Builder functions

[martinbonnin]

Deprecate things like

ApolloClient.Builder()
    .httpEngine(...)
    .webSocketTimeoutMillis(...)
    // etc...
    .build()

Those are properties of HttpNetworkTransport and WebSocketNetworkTransport respectively and what we gain in the "simple" cases is outweight by the complexity induced (more docs, impossible states, bad error messages (#6253), etc...).

I am now convinced that precedence rules are clumsy, and this PR removes a lot of the precedence.

The code above can be replaced by the (longer but also much more consistent and less footgunny) version:

ApolloClient.Builder()
    .networkTransport(HttpNetworkTransport.Builder().httpEngine(engine).build())
    .subscriptionNetworkTransport(WebSocketNetworkTransport.Builder().timeoutMillis(timeoutMillis).build())
    // etc...
    .build()

I just kept serverUrl() because it's used everywhere in the docs.

[svc-apollo-docs]

✅ Docs Preview Ready

No new or changed pages found.

[martinbonnin]

New constructor. We need that to avoid initializing from the main thread

[BoD]

More explicit > less explicit 👍

[BoD]

I worry a bit about this one in particular as it's probably one the most often used, and the replacement expression is a bit of mouthful... But I agree about the tradeoff, it's for the better.

[martinbonnin]

That's a fair point. We don't want to become the capitalized() of GraphQL... Will think more about this