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.

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

Versions update & use API key. #13

Merged
merged 5 commits into from
Sep 13, 2017
Merged

Conversation

rok-povsic
Copy link
Contributor

I updated all the library dependencies, including the Clarifai Java client, to the latest version.

I also switched from using app ID/secret to API key.

Closes #12.

Copy link

@josh-tepper josh-tepper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Just be sure to also change the "Building and Running" section of clarifai-android-starter/README.md before submitting

@josh-tepper
Copy link

Also, travis is failing with:

The command "echo no | android create avd --force -n test -t android-18 --abi armeabi-v7a" failed and exited with 1 during .
Your build has been stopped.

You might need to update: clarifai-android-starter/.travis.yml

@eddiezane eddiezane force-pushed the updates branch 4 times, most recently from b3b195b to 9e09e09 Compare September 13, 2017 21:49
@eddiezane eddiezane merged commit 0650db4 into Clarifai:master Sep 13, 2017
@rok-povsic rok-povsic deleted the updates branch September 14, 2017 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants