Skip to content
This repository was archived by the owner on Dec 7, 2019. It is now read-only.

Updated dependencies.#38

Merged
yunikkk merged 2 commits intogojuno:masterfrom
yunikkk:update-dependencies
May 14, 2018
Merged

Updated dependencies.#38
yunikkk merged 2 commits intogojuno:masterfrom
yunikkk:update-dependencies

Conversation

@yunikkk
Copy link
Copy Markdown
Contributor

@yunikkk yunikkk commented May 13, 2018

Resolves #37

@yunikkk yunikkk requested a review from artem-zinnatullin May 13, 2018 19:41
@yunikkk yunikkk force-pushed the update-dependencies branch from bac30b8 to 9dfe97e Compare May 13, 2018 19:42
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-3.4-bin.zip
distributionUrl=https\://services.gradle.org/distributions/gradle-4.6-bin.zip
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

pls run something with Gradle to update the wrapper, also you can use 4.7

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Well, wrapper isn't updating, Ive ran it many times...

--env BINTRAY_USER="$BINTRAY_USER" \
--env BINTRAY_API_KEY="$BINTRAY_API_KEY" \
--env BINTRAY_GPG_PASSPHRASE="$BINTRAY_GPG_PASSPHRASE" \
--env ANDROID_HOME="" \
Copy link
Copy Markdown
Contributor Author

@yunikkk yunikkk May 14, 2018

Choose a reason for hiding this comment

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

Sad workaround for NPE when accessing adb in tests, with new commander
when ANDROID_HOME is not set.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Or we could wrap them locally...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ouch, well ok for now I guess

--env BINTRAY_USER="$BINTRAY_USER" \
--env BINTRAY_API_KEY="$BINTRAY_API_KEY" \
--env BINTRAY_GPG_PASSPHRASE="$BINTRAY_GPG_PASSPHRASE" \
--env ANDROID_HOME="" \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ouch, well ok for now I guess

@yunikkk yunikkk merged commit 2342a6f into gojuno:master May 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants