Skip to content

Convert Plaid to use AndroidX #524

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.

Sign up for GitHub

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 17 commits into from
Oct 21, 2018
Merged

Convert Plaid to use AndroidX #524

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
dd2ebf7
Initial commit after AndroidX conversion using Android Studio refacto…
tiembo Oct 12, 2018
6b23efa
Bump Glide to 4.8.0, which supports AndroidX annotations
tiembo Oct 12, 2018
8e60a35
Fix dex error with PathSegment
tiembo Oct 12, 2018
75b8ffd
Fix error with type mismatch
tiembo Oct 12, 2018
d2cefa3
Temporarily comment out plusAssign usage. App runs now
tiembo Oct 12, 2018
3a038ef
Clean up imports
tiembo Oct 12, 2018
aba91a9
Attempt to fix core-runtime CI error
tiembo Oct 12, 2018
b38f2cf
Upgrade Kotlin and Gradle wrapper versions
tiembo Oct 12, 2018
0c5a3d6
Extract versions out of build.gradle files
tiembo Oct 12, 2018
44dc2ba
Address PR comments from Nick
tiembo Oct 17, 2018
0a5a5a3
Address PR comment from Florina
tiembo Oct 17, 2018
a520c32
Rename version variable names
tiembo Oct 17, 2018
7aad300
Update palette params for applyFullImage and applyTopPalette
tiembo Oct 17, 2018
27fbae9
Fix format violations
tiembo Oct 17, 2018
9715e2f
Workaround for dependency conflict during assembleAndroidTest
tiembo Oct 17, 2018
a997200
Fix instrumentation tests
tiembo Oct 21, 2018
4109a57
Update Kotlin to 1.3.0-rc-190
tiembo Oct 21, 2018
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Attempt to fix core-runtime CI error
  • Loading branch information
@tiembo
tiembo committed Oct 12, 2018
commit aba91a9cd5a7a92dc5b9863a6b8c9f980597726b
5 changes: 5 additions & 0 deletions designernews/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,10 @@ android {
enabled true
}

dependencies {
Copy link
Owner

Choose a reason for hiding this comment

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

Shouldn't this be in the other deps block below?
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes - moved.
implementation("androidx.arch.core:core-runtime:2.0.1-alpha01")
}

compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
Expand All @@ -52,6 +56,7 @@ dependencies {
implementation project(':app')

testImplementation project(':test_shared')

testImplementation 'androidx.arch.core:core-testing:2.0.0'
testImplementation "com.nhaarman.mockitokotlin2:mockito-kotlin:${versions.mockito_kotlin}"
testImplementation "junit:junit:${versions.junit}"
Expand Down