add url support as optional argument to make_client

[iamsudip]

No description provided.

[codecov]

Codecov Report

Merging #80 into master will increase coverage by 0.17%.
The diff coverage is 74.07%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   80.61%   80.79%   +0.17%     
==========================================
  Files          35       35              
  Lines        3493     3540      +47     
==========================================
+ Hits         2816     2860      +44     
- Misses        677      680       +3

Impacted Files	Coverage Δ
thriftpy2/http.py	78.73% <100%> (+5.27%)	⬆️
thriftpy2/tornado.py	82.6% <63.63%> (-1.4%)	⬇️
thriftpy2/rpc.py	81.69% <64.7%> (-2.53%)	⬇️
thriftpy2/contrib/aio/rpc.py	83.33% <66.66%> (-4.17%)	⬇️
thriftpy2/thrift.py	77.06% <0%> (+0.71%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 90cbc37...75d8fd0. Read the comment docs.

[ethe]

I think it is better to change the protocol type in the URL, https:// and tcp:// do not make sense. In my opinion, using thrift:// is better because of the usual practice.

[iamsudip]

I think it is better to change the protocol type in the URL, https:// and tcp:// do not make sense. In my opinion, using thrift:// is better because of the usual practice.

Agree. Let me change that.

[iamsudip]

@ethe Have a look now! Made the changes and added more tests to increase coverage ;)

[ethe]

Merged, thank you.