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

Updated dependencies. #38

Merged
merged 2 commits into from
May 14, 2018
Merged

Updated dependencies. #38

merged 2 commits into from
May 14, 2018

Conversation

yunikkk
Copy link
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
@@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
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
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
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...

@@ -36,9 +36,10 @@ docker run \
--env BINTRAY_USER="$BINTRAY_USER" \
--env BINTRAY_API_KEY="$BINTRAY_API_KEY" \
--env BINTRAY_GPG_PASSPHRASE="$BINTRAY_GPG_PASSPHRASE" \
--env ANDROID_HOME="" \
Copy link
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
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
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

@@ -36,9 +36,10 @@ docker run \
--env BINTRAY_USER="$BINTRAY_USER" \
--env BINTRAY_API_KEY="$BINTRAY_API_KEY" \
--env BINTRAY_GPG_PASSPHRASE="$BINTRAY_GPG_PASSPHRASE" \
--env ANDROID_HOME="" \
Copy link
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