Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

A few minor changes #9

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

A few minor changes #9

wants to merge 4 commits into from

Commits on Apr 12, 2020

  1. Make the code more portable by: 

    * Removing client_id.json from repo and rely on file path passed as --auth option, with a default value of client_secret.json. The credentials file shouldn't be part of the repo, because it will always require manual changing (doesn't like it), and it can be outdated (as example, the token_uri url was wrong in the existent client_id.json file).
* Passing the credentials file path to auth function, instead of using the hardcoded client_id.json value
* Apply PEP8 formatting to upload.py file
    @jorgeas80
    jorgeas80 committed Apr 12, 2020
    Configuration menu
    Copy the full SHA
    b3db2b9 View commit details
    Browse the repository at this point in the history

  2. Update README with the right instructions

    @jorgeas80
    jorgeas80 committed Apr 12, 2020
    Configuration menu
    Copy the full SHA
    7eae00e View commit details
    Browse the repository at this point in the history

  3. Allow directory as input instead of a list of image files

    @jorgeas80
    jorgeas80 committed Apr 12, 2020
    Configuration menu
    Copy the full SHA
    38f58d7 View commit details
    Browse the repository at this point in the history

  4. Updated README and help comments

    @jorgeas80
    jorgeas80 committed Apr 12, 2020
    Configuration menu
    Copy the full SHA
    d81e79b View commit details
    Browse the repository at this point in the history