TryUpload() should be named Upload()

simonedegiacomi / gphotosuploader

Unofficial Google Photos uploader and Go library

GNU General Public License v3.0

99 stars 29 forks source link

TryUpload() should be named Upload() #8

Closed nmrshll closed 6 years ago

nmrshll commented 6 years ago

It returns an error if it fails anyway.

Of course, there's already an upload() method, but maybe we could do some refactoring and put the 3 upload steps (get uploadID, upload, enable) in a sub-package named uploadSteps ?

That way the top level method that does these 3 upload steps can be called Upload() too.