-
Notifications
You must be signed in to change notification settings - Fork 391
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 updated; deprecated configurations updated. #31
base: master
Are you sure you want to change the base?
Conversation
@@ -1,4 +1,4 @@ | |||
distributionUrl=https\://services.gradle.org/distributions/gradle-4.5.1-bin.zip | |||
distributionUrl=https\://services.gradle.org/distributions/gradle-4.6-bin.zip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason to stay so low? 4.10.2 is latest
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's just the current "least" version for build-tools 3.2.0.
….files() instead of TaskInputs.file() ).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
@@ -3,3 +3,4 @@ | |||
/out | |||
/.idea | |||
/.gradle | |||
/local.properties |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Move before the "/out" to keep in alphabetical order (although I realize now that the gradle is incorrectly placed...)
compileWindows "com.android.tools.build:aapt2:3.2.0-alpha14-4748712:windows" | ||
compileMacOs "com.android.tools.build:aapt2:3.2.0-alpha14-4748712:osx" | ||
compileLinux "com.android.tools.build:aapt2:3.2.0-alpha14-4748712:linux" | ||
compileWindows "com.android.tools.build:aapt2:3.2.0:windows" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the stable version is: 3.2.0-4818971
shadow "com.google.errorprone:error_prone_annotations:2.2.0" | ||
shadow "com.google.guava:guava:26.0-jre" | ||
shadow "com.google.protobuf:protobuf-java:3.4.0" | ||
shadow "com.google.protobuf:protobuf-java-util:3.4.0" | ||
shadow "com.google.protobuf:protobuf-java:3.6.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you could switch to 3.6.0 instead, that would be helpful. We can't use 3.6.1 on our internal branch at the moment.
Updates all yaml files to correct github path.
No description provided.