search

Julian-O / Instagram-API-python

Unofficial Instagram API in Python.
Other
22 stars 15 forks source link

fix photo upload and pep8 sytle #3

Closed nyx-rattapoom closed 7 years ago

nyx-rattapoom commented 7 years ago

fix mistake line of code and incorrect variable name fix missing api url pep8 sytle more compilable with python3

Julian-O commented 7 years ago

I wouldn't have chosen to change the filenames to include underscores.

Here's what PEP8 says:

Modules should have short, all-lowercase names. Underscores can be used in the module name if it improves readability. Python packages should also have short, all-lowercase names, although the use of underscores is discouraged.

However, I am happy to proceed with the change. If someone wants to change it back, I'd probably accept that too.

Julian-O commented 7 years ago

Thanks for the changes. Note that I hope to make some more client-breaking changes in the near future - particularly changing the return types of the endpoint calls.