-
Notifications
You must be signed in to change notification settings - Fork 515
Migrate to androix #47
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
Conversation
8c4aaf4 to
de2bb01
Compare
Codecov Report
@@ Coverage Diff @@
## master #47 +/- ##
=========================================
Coverage ? 53.62%
Complexity ? 19
=========================================
Files ? 9
Lines ? 69
Branches ? 3
=========================================
Hits ? 37
Misses ? 28
Partials ? 4
Continue to review full report at Codecov.
|
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 am not owner of this repository, but as a developer I wouldn't add any new libraries to any merge request without a note why it is important to this request or repository :-)
| testImplementation cacheTestDependencies.junit | ||
| testImplementation cacheTestDependencies.kotlinJUnit | ||
| testImplementation cacheTestDependencies.mockito | ||
| testImplementation cacheTestDependencies.kotlinReflect |
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 do not think that this library is necessary. Just curious - why did you add it in every module?
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.
This is so long ago, I'm not sure any more... Maybe it has to do with a gradle warning saying that different versions of a lib were found on the classpath. Maybe it's something else. Feel free to remove it. If it works well then I guess it's not needed !
| testingSupportLibVersion = '0.1' | ||
| mockitoKotlinVersion = '1.5.0' | ||
| mockitoAndroidVersion = '2.8.47' | ||
| mockitoAndroidVersion = '2.8.9' // As of today, 2.23.0 does not work |
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.
downgraded library?
| androidAnnotations: "androidx.annotation:annotation:1.0.0", | ||
| appCompat: "androidx.appcompat:appcompat:${appCompatVersion}", | ||
| supportRecyclerView:"androidx.recyclerview:recyclerview:1.0.0", | ||
| supportDesign: "com.google.android.material:material:1.0.0-rc01", |
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 am pretty sure that when you use supportDesign library you don't need to add supportRecyclerView.
| @@ -1,3 +1,7 @@ | |||
| plugins { | |||
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.
What is purpose of this plugin? Is it related to Migrate to androidX MR?
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.
It's used to check if new versions of libraries are available. No needed for androidx. You can skip it.
|
Closing due to inactivity |
This PR migrates to AndroidX and Android Studio 3.2.1.
It also adds the gradle-versions-plugin and bumps several libraries to their latest version