chore(android): Bump version for Next storage

[krizzu]

Summary

• Bumped default Kotlin version to 1.9.20
• Coroutines bumped to 1.7.3
• Dropped kapt in favor of ksp, which version can be set via gradle property

[tido64]

Will this cause issues when react-native is still on 1.8? And what about older versions?

In react-native-test-app, we keep a list of compatible versions: https://github.com/microsoft/react-native-test-app/blob/e43b6a622bffaa76ce28e1ed1c4f036db6611342/android/dependencies.gradle#L6

Should we be doing the same here?

[krizzu]

Kotlin is being kept forward binary compatible, so It should be fine with older versions. Gradle will select latest kotlin version and evict the rest.

I do like your suggestion tho, I could add a map of supported KSP versions, so we can decide on it based on Kt version used

[leofolive]

The addition of the supported KSP versions map will be particularly valuable for legacy projects, providing a clear understanding of compatibility.

[krizzu]

@tido64 Moved the config to separate file, for easier management. I left an option to still be able to override KSP version, in case Kotlin v2 and supporting KSP releases, before we add support for it.

[krizzu]

@tido64 CI is failing due to

Plugin [id: 'com.google.devtools.ksp', version: 'null'] was not found in any of the following sources

I had same issue locally, where dependencies.gradle file could not be read from TestApp.

[tido64]

@tido64 CI is failing due to

Plugin [id: 'com.google.devtools.ksp', version: 'null'] was not found in any of the following sources

I had same issue locally, where dependencies.gradle file could not be read from TestApp.

Seems like we're missing the latest version: microsoft/react-native-test-app#1701

I've also updated the logic to not return null.

Update: Can you try bumping to 2.5.33?

[krizzu]

Bumped, let's give it a go 🤞

[tido64]

Looks like we need to bump Robolectric:

com.reactnativecommunity.asyncstorage.next.AsyncStorageAccessTest > initializationError FAILED
    java.lang.IllegalArgumentException: failed to configure com.reactnativecommunity.asyncstorage.next.AsyncStorageAccessTest.performsBasicGetSetRemoveOperations: Package targetSdkVersion=33 > maxSdkVersion=31
        at org.robolectric.RobolectricTestRunner.getChildren(RobolectricTestRunner.java:260)

Latest version should support up to 34.

[krizzu]

Yeah, will try to do it later day or tomorrow 🙏
Done

[krizzu]

for some weird reason, the src dir was not added, so had to add all and exclude build files

[tido64]

This might be related to the rewrite of the glob implementation in npm 9. We've hit issues with it as well: npm/npm-packlist#152

[krizzu]

not cool!

[tido64]

🚀

[krizzu]

@tido64 added a log to it. I'm going to release this as minor version update, due to libraries bump

[AsyncStorageBot]

🎉 This PR is included in version 1.20.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀