Skip to content

[shared_preferences] Platform implementations of async api #6965

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
auto-submit merged 23 commits into from
Jul 30, 2024
Merged

[shared_preferences] Platform implementations of async api #6965

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
0b619b4
plat
tarrinneal Jul 18, 2024
b8b4643
analyze
tarrinneal Jul 18, 2024
3216359
newer version
tarrinneal Jul 18, 2024
351e189
revert gradle changes
tarrinneal Jul 18, 2024
167a191
revert gradle wrapper
tarrinneal Jul 19, 2024
751b7ab
gradle
tarrinneal Jul 19, 2024
966cc82
bom
tarrinneal Jul 19, 2024
f5a7965
move bom
tarrinneal Jul 19, 2024
6d2127b
new kotlin version
tarrinneal Jul 20, 2024
5e80acb
rename classes and files
tarrinneal Jul 22, 2024
6a148c4
legacy
tarrinneal Jul 22, 2024
f3002f1
more legacy
tarrinneal Jul 22, 2024
48a294f
one more legacy
tarrinneal Jul 23, 2024
9d4ce28
anotha one
tarrinneal Jul 23, 2024
2b7b3a6
fix tests
tarrinneal Jul 23, 2024
fa7389d
revert test fix, fix real issue
tarrinneal Jul 23, 2024
bc85c30
fix explode
tarrinneal Jul 24, 2024
9780d68
format
tarrinneal Jul 24, 2024
9249648
revert kotlin version, update changelog
tarrinneal Jul 25, 2024
475237f
bom
tarrinneal Jul 26, 2024
94498a0
Merge branch 'main' of github.com:flutter/packages into spr-plat
tarrinneal Jul 26, 2024
78ace3a
move bom
tarrinneal Jul 26, 2024
89ed80a
kotlin version, how do they work?
tarrinneal Jul 27, 2024
File filter

Filter by extension

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
new kotlin version
  • Loading branch information
@tarrinneal
tarrinneal committed Jul 20, 2024
commit 6d2127b062351c9260ee4e365c33a4fdaf2da989
7 changes: 2 additions & 5 deletions packages/shared_preferences/shared_preferences_android/android/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ group 'io.flutter.plugins.sharedpreferences'
version '1.0-SNAPSHOT'

buildscript {
ext.kotlin_version = '1.7.10'
ext.kotlin_version = '1.8.10'
Copy link
Collaborator

@stuartmorgan-g stuartmorgan-g Jul 25, 2024

Choose a reason for hiding this comment

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

@reidbaker Is this going to cause issues for clients? It looks like this would be the first of our plugins using 1.8.x.

@tarrinneal What exactly was the error when this was 1.7.10 like our other plugins?

Copy link
Contributor Author

@tarrinneal tarrinneal Jul 25, 2024

Choose a reason for hiding this comment

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

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8741408197539124465/+/u/Run_package_tests/native_unit_tests/stdout?format=raw

Integration tests run fine though, only unit tests fail. There may be some other underlying issue.

repositories {
google()
mavenCentral()
Expand Down Expand Up @@ -61,10 +61,7 @@ android {
warningsAsErrors true
disable 'AndroidGradlePluginVersion', 'InvalidPackage', 'GradleDependency', 'NewerVersionAvailable'
}
dependencies {
// org.jetbrains.kotlin:kotlin-bom artifact purpose is to align kotlin stdlib and related code versions.
// See: https://youtrack.jetbrains.com/issue/KT-55297/kotlin-stdlib-should-declare-constraints-on-kotlin-stdlib-jdk8-and-kotlin-stdlib-jdk7
implementation(platform("org.jetbrains.kotlin:kotlin-bom:1.8.10"))
dependencies {
implementation 'androidx.datastore:datastore:1.0.0'
implementation 'androidx.datastore:datastore-preferences:1.0.0'
testImplementation 'junit:junit:4.13.2'
Expand Down
2 changes: 1 addition & 1 deletion packages/shared_preferences/shared_preferences_android/example/android/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
buildscript {
ext.kotlin_version = '1.7.10'
ext.kotlin_version = '1.8.10'
repositories {
google()
mavenCentral()
Expand Down