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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgraded project to gradle 5.1.1 and kotlin 1.3.11 #212
Conversation
leku/build.gradle
Outdated
@@ -139,7 +177,7 @@ bintray { | |||
description = "Location picker component for Android. It returns a latitude,longitude and an address based on the location picked in the LocationPickerActivity provided." | |||
publish = true | |||
dryRun = false | |||
override = true | |||
override = false |
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.
Could you say why?
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.
Because we don't want to override an AAR already published.
This is what Docu says:
override = false //[Default: false] Whether to override version artifacts already published
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.
@CristianGM please check it out again. Thanks!
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.
Then it should have never been true.
It would be better to remove the line and keep it default.
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.
@CristianGM done! You're right it never should have been true but it was hard to make it work without testing these things.
This PR tries to solve one of the biggest issues of the recent versions. After ditching the old Novoda bintray plugin we had serious problems for uploading an AAR with all dependencies and sources. So I decided to upgrade gradle, all dependencies and add the sources to the publishing task. Hopefully this will solve the issue. 馃檹