sharat87/requests — comment on commit 54242833c7e8bf0949908bf44a0ec092ae780463
File:  line 0
By sharat87:

Sure, I agree they are optional dependencies. But, what I'm saying is one should be able to just clone, get deps, run tests, modify code, run tests. But currently, the tests step fails, and one has to look at the tests code to figure out that there is a dependency missing for the tests to run. This is not a very welcoming experience, especially when this is not documented anywhere (is it? I couldn't find it).

That's my opinion, which is why I created this pull request. And yeah, if envoy is an unused import, it can be removed.