Skip to content
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

Update dependencies to stay modern #39

Merged
merged 3 commits into from
Aug 9, 2022

Conversation

jahunt1
Copy link
Contributor

@jahunt1 jahunt1 commented Aug 9, 2022

Highlights:

  • Updated Kotlin version to 1.7.0
  • Updated AGP to version 7.2.2
  • Updated Gradle wrapper to 7.5.1

api "androidx.annotation:annotation:1.2.0"
compileOnly 'io.intercom.android:intercom-sdk-base:10.6.1'
api "androidx.annotation:annotation:$rootProject.ext.annotationsVersion"
compileOnly 'io.intercom.android:intercom-sdk-base:12.0.0'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is a newer version of the Intercom SDK available, but updating to the latest version will require updating more test code than I was willing to touch in this PR. Let's update this SDK version in a follow-up PR.

api "androidx.annotation:annotation:1.2.0"
compileOnly 'com.kissmetrics.sdk:KISSmetricsSDK:2.3.1'
api "androidx.annotation:annotation:$rootProject.ext.annotationsVersion"
// compileOnly 'com.kissmetrics.sdk:KISSmetricsSDK:2.3.1' // Not working
Copy link
Contributor Author

@jahunt1 jahunt1 Aug 9, 2022

Choose a reason for hiding this comment

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

The latest versions of the KISSmetrics SDK were on jcenter() which has been sunsetted. As a temporary (I hope 🤞 ) workaround I forked their SDK repo and made it so Jitpack could build the latest version.

Copy link
Contributor

Choose a reason for hiding this comment

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

That's too bad.
Is their library deprecated? Or they just haven't gotten around to moving it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems like they haven't paid much attention to their SDK if they haven't made the time to migrate to a different public repository. I raised a PR in case they want to be able to use Jitpack to build their binaries. 🤷‍♂️

Comment on lines 53 to 55
compileOnly 'com.google.android.gms:play-services-analytics:17.0.0'

testImplementation 'com.google.android.gms:play-services-analytics:17.0.0'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is a newer version of the Google Analytics SDK too, but I think there were some breaking changes and I would have had to conform to different function signatures. Let's update to the latest SDK in a follow-up PR.

@jahunt1 jahunt1 requested a review from simon1867 August 9, 2022 06:32
Copy link
Contributor

@simon1867 simon1867 left a comment

Choose a reason for hiding this comment

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

Looks good to me 😄

api "androidx.annotation:annotation:1.2.0"
compileOnly 'com.kissmetrics.sdk:KISSmetricsSDK:2.3.1'
api "androidx.annotation:annotation:$rootProject.ext.annotationsVersion"
// compileOnly 'com.kissmetrics.sdk:KISSmetricsSDK:2.3.1' // Not working
Copy link
Contributor

Choose a reason for hiding this comment

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

That's too bad.
Is their library deprecated? Or they just haven't gotten around to moving it?

@jahunt1 jahunt1 merged commit fd95389 into alignops:master Aug 9, 2022
@jahunt1 jahunt1 deleted the dependency-update-aug-2022 branch August 9, 2022 16:29
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.

2 participants