Question: Patron and streaming

[jcoyne]

It has been explained to me in #620 that streaming is scheduled for 1.0, and that all adapters must be made to support it. However, Patron does not support streaming (toland/patron#139). Should we:

1. Drop Patron as a supported adapter
2. Change the expectation that all features must be supported by all drivers

[iMacTia]

@jcoyne I can see your point so let me rephrase what I said in #620 (comment).

We request contributors of new features to try making their features supported by all adapters. If an adapter is not capable of supporting that feature at all then, considering the importance of the feature and the interest from the community, the feature might still be added without support for that adapter.
This is a case-by-case decision, however I can see streaming being an important feature for the community, so this can be accepted if supported by most (but not all) adapters.

Again, this is clearly stated also in #485:

I have no experience with SOCKS, but to add a feature to an adapter in Faraday, the attempt must be made to add the same feature to all other adapters as well. Think you can look into it?

[jcoyne]

@iMacTia Thanks for the clarification. Perhaps this policy could be moved into a README or CONTRIBUTING guide?

[iMacTia]

Totally agree with that. I just checked the CONTRIBUTING guide being sure I would find it, but I notice it was missing from there!
I'll add it straight away so hopefully I don't have to look for those comments again in the future

[iMacTia]

https://github.com/lostisland/faraday/blob/master/CONTRIBUTING.md#new-features