search

InCodeLearning / InCodeLearning-Python2

1 stars 0 forks source link

speedtest-cli #5

Closed JesseZhuang closed 8 years ago

JesseZhuang commented 8 years ago

interesting http tool working with speedtest.net API.

ywang0701 commented 8 years ago

http://www.speedtest.net/ is very impressive on measuring upload/download speed and give you your IP, location, and internet provider. My download/upload speed is 58.43/6.06 Mbps. The UI with a global map makes the tool more user friendly.

This is a great tool to keep in the tool box and a clean code to learn from.

JesseZhuang commented 8 years ago

@ywang0701 exactly, look here. I will assign this to both of us.

ywang0701 commented 8 years ago

@JesseZhuang The code works like a charm. I propose to push this PR to dev branch and create a master-PR. Then, you can push it to master branch.

ywang0701 commented 8 years ago

FYI, I don't have the right to merge this PR. So, either you give me the merge-permission or you can merge this PR to dev. Then you can create its master-PR and execute the merge. Thx -Yong

image

JesseZhuang commented 8 years ago

We will first discuss all PRs on weekly meeting before we merge them. That's the workflow.