Skip to content

Commit

Permalink
Upgrade compose version to match compiler version (#353)
Browse files Browse the repository at this point in the history
* Also upgrade compose version to match compiler version

* Also upgrade compileSdkVersion to 33

* Fix another build failure

* Same build error

* Fix failing tests

* Fix lint failure

* Upgrade to Java 17

* More changes to fix build errors
  • Loading branch information
vinaygaba authored Sep 19, 2023
1 parent 360259e commit 5df9543
Show file tree
Hide file tree
Showing 117 changed files with 122 additions and 111 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/android.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,10 @@ jobs:

steps:
- uses: actions/checkout@v2
- name: set up JDK 11
- name: set up JDK 17
uses: actions/setup-java@v1
with:
java-version: 11
java-version: 17
- uses: actions/cache@v1
with:
path: ~/.gradle/caches
Expand All @@ -31,10 +31,10 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: set up JDK 11
- name: set up JDK 17
uses: actions/setup-java@v1
with:
java-version: 11
java-version: 17
- name: Checkout Branch
uses: actions/checkout@v2
- uses: actions/cache@v1
Expand Down Expand Up @@ -71,10 +71,10 @@ jobs:
target: [ default, google_apis ]
steps:
- uses: actions/checkout@v2
- name: set up JDK 11
- name: set up JDK 17
uses: actions/setup-java@v1
with:
java-version: 11
java-version: 17
- name: Checkout Branch
uses: actions/checkout@v2
- uses: actions/cache@v1
Expand Down
6 changes: 3 additions & 3 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,14 @@ buildscript {
'androidXTestCore' : '1.4.0',
'androidXTestRules' : '1.4.0',
'assertJ' : '3.16.1',
'compose' : '1.2.1',
'compose' : '1.4.2',
'composeCompiler' : '1.4.2',
'composeActivity' : '1.5.1',
'composeConstraintLayout': '1.0.1',
'composeNavigation' : '2.5.1',
'detekt' : '1.7.4',
'espresso' : '3.2.0',
'gradle' : '7.4.0',
'gradle' : '8.0.0',
'junit' : '4.13.2',
'junitImplementation' : '1.1.2',
'kotlin' : '1.8.10',
Expand All @@ -35,7 +35,7 @@ buildscript {
'strikt' : '0.33.0',
'xprocessing' : '2.6.0-alpha01',
'corektx' : '1.7.0',
'shot' : '5.13.0'
'shot' : '6.0.0'
]
ext.deps = [
'compose' : [
Expand Down
4 changes: 2 additions & 2 deletions gradle/wrapper/gradle-wrapper.properties
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#Fri Aug 07 16:12:58 PDT 2020
#Tue Sep 19 07:58:34 PDT 2023
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-8.0-bin.zip
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-7.6.1-bin.zip
11 changes: 6 additions & 5 deletions sample-submodule-2/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,12 @@ if (project.hasProperty('useKsp')) {
}

android {
compileSdkVersion 32
namespace "com.airbnb.android.submodule.showkasesample"

defaultConfig {
minSdkVersion 21
targetSdkVersion 32
compileSdk 33
targetSdkVersion 33

testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
}
Expand All @@ -32,11 +33,11 @@ android {
}
}
compileOptions {
sourceCompatibility JavaVersion.VERSION_11
targetCompatibility JavaVersion.VERSION_11
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
}
kotlinOptions {
jvmTarget = '11'
jvmTarget = '17'
}
buildFeatures {
compose true
Expand Down
2 changes: 1 addition & 1 deletion sample-submodule-2/src/main/AndroidManifest.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<?xml version="1.0" encoding="utf-8"?>
<manifest package="com.airbnb.android.second.submodule">
<manifest>

</manifest>
11 changes: 6 additions & 5 deletions sample-submodule/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,12 @@ if (project.hasProperty('useKsp')) {
}

android {
compileSdkVersion 32
namespace "com.airbnb.android.submodule.showkasesample"

defaultConfig {
minSdkVersion 21
targetSdkVersion 32
compileSdk 33
targetSdkVersion 33

testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
}
Expand All @@ -33,11 +34,11 @@ android {
}
}
compileOptions {
sourceCompatibility JavaVersion.VERSION_11
targetCompatibility JavaVersion.VERSION_11
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
}
kotlinOptions {
jvmTarget = '11'
jvmTarget = '17'
}
buildFeatures {
compose true
Expand Down
3 changes: 1 addition & 2 deletions sample-submodule/src/main/AndroidManifest.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.airbnb.android.submodule">
<manifest xmlns:android="http://schemas.android.com/apk/res/android">

</manifest>
11 changes: 6 additions & 5 deletions sample/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,13 @@ if (project.hasProperty('useKsp')) {
}

android {
compileSdkVersion 32
namespace "com.airbnb.android.showkasesample"

defaultConfig {
applicationId "com.airbnb.android.showkasesample"
minSdkVersion 21
targetSdkVersion 32
compileSdk 33
targetSdkVersion 33
versionCode 1
versionName "1.0"
testInstrumentationRunner "com.karumi.shot.ShotTestRunner"
Expand All @@ -38,11 +39,11 @@ android {
}

compileOptions {
sourceCompatibility = JavaVersion.VERSION_11
targetCompatibility = JavaVersion.VERSION_11
sourceCompatibility = JavaVersion.VERSION_17
targetCompatibility = JavaVersion.VERSION_17
}
kotlinOptions {
jvmTarget = '11'
jvmTarget = '17'
}
buildFeatures {
compose true
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
{"screenshots":[{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_0","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_0"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_1","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_1"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_2","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_2"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_3","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_3"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_4","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_4"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_5","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_5"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_6","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_6"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_7","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_7"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_8","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_8"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_9","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_9"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_10","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_10"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_11","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_11"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_12","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_12"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_10","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_10"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_11","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_11"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_12","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_12"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_13","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_13"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_14","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_14"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_15","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_15"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_16","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_16"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_17","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_17"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_18","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_18"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_composable_screenshot_test_19","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"composable_screenshot_test_19"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_0","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_0"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_1","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_1"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_2","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_2"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_3","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_3"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_4","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_4"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_5","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_5"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_6","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_6"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_color_screenshot_test_7","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"color_screenshot_test_7"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_0","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_0"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_1","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_1"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_2","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_2"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_3","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_3"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_4","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_4"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_5","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_5"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_6","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_6"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_7","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_7"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_8","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_8"},{"name":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen_typography_screenshot_test_9","testClassName":"com.airbnb.android.showkasesample.MyScreenshotTest_ShowkaseCodegen","testName":"typography_screenshot_test_9"}]}
Loading

0 comments on commit 5df9543

Please sign in to comment.