Skip to content

Dependency updates #105

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

Merged
merged 1 commit into from
Dec 4, 2019
Merged

Dependency updates #105

merged 1 commit into from
Dec 4, 2019

Conversation

Jawnnypoo
Copy link
Member

Title says it!

@Jawnnypoo Jawnnypoo requested a review from rogerhu December 4, 2019 15:50
classpath 'com.github.dcendents:android-maven-gradle-plugin:2.1'
}
}

plugins {
id 'com.github.ben-manes.versions' version '0.20.0'
id "com.github.ben-manes.versions" version "0.27.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

fyi - i think we can get rid of this now that Gradle/Android Studio comes with built-in version checking

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah I've had mixed results with that. It works generally, but also sometimes misses dependencies that have not been fetched already by other projects or manually via this plugin.

api "com.github.parse-community.Parse-SDK-Android:parse:1.19.0"
api 'com.squareup.okhttp3:okhttp:3.12.1'
api "com.github.parse-community.Parse-SDK-Android:parse:1.22.1"
api "com.squareup.okhttp3:okhttp:3.14.4"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there something about OkHttp 3.14.4 that was necessary for LiveQuery to be updated?

Note that Parse-SDK-Android explicitly adds a note that 3.13 and newer should be avoided, because OkHttp 3.13 bumps the minSdKVersion to Android 5.0: https://github.com/parse-community/Parse-SDK-Android/blob/master/parse/build.gradle#L31-L32

If 3.14 isn't an absolute requirement for something, I think LiveQuery should follow the same reasoning here, and stick to 3.12. Otherwise line 8 minSdkVersion 15 is wrong, or at the very least it's misleading.

I opened #113 to track this.

@mtrezza mtrezza deleted the updates branch December 22, 2022 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants