conform to pep8 guidelines

[SanketDG]

Hey just fixed your code to conform to pep8.

Thanks!

[harshasrinivas]

make sure that the __future__ import lines are placed at the beginning of the file.

[SanketDG]

Yeah, ok. Done!

You should turn off Travis CI, you have no tests.

[harshasrinivas]

Cool. Just put these two lines on top as well.
from future.standard_library import install_aliases
install_aliases()

It is used to make the module urllib.request compatible with python 2.x!

[harshasrinivas]

And thanks for fixing the typo! 👍

[SanketDG]

It is used to make the module urllib.request compatible with python 2.x!

Oh! I see. Sorry for that.
btw I removed from string import * is that okay?

[harshasrinivas]

yes, string module is important. it is what is responsible for the find() method to work

[SanketDG]

Well, have you every tried using find without importing string module? It works. Every string object by default has these methods, you don't need to import anything for them to work.

[sananth12]

@harshasrinivas
Also, avoid import * at all costs.

https://www.python.org/dev/peps/pep-0020/

Explicit is better than implicit.

[harshasrinivas]

yeah thanks @sananth12 ! 👍
@SanketDG oh cool, thanks! You can make the remaining changes now :)

[SanketDG]

Ready for merge.

[harshasrinivas]

Thanks! @SanketDG , you have been added to the list of contributors 😅