Bump kotlinx.collections.immutable to 0.5.0-beta01#6919
Conversation
Applied via kotlin-tooling-immutable-collections-0-5-0-migration skill. 0.5.0-beta01 is binary-compatible with 0.4.0; per KEEP-0459 the copy-returning methods on Persistent* types are renamed (e.g. add->adding, remove->removing) and the old names remain as @deprecated(WARNING) overloads. bitwarden-android uses the library as a read-only consumer (persistentListOf -> ImmutableList<T> typed values, no in-place mutation via Persistent* receivers), so the compiler emits zero deprecation warnings post-bump and no call sites need to be renamed. See MIGRATION_REPORT.md for per-phase details. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Thank you for your contribution! We've added this to our internal Community PR board for review. Details on our contribution process can be found here: https://contributing.bitwarden.com/contributing/pull-requests/community-pr-process. |
|
Thank you for your contribution! We've added this to our internal Community PR board for review. Details on our contribution process can be found here: https://contributing.bitwarden.com/contributing/pull-requests/community-pr-process. |
|
@DmitryNekrasov Is there a particular reason we need to jump to the beta? We usually try to stick to stable releases unless there is a good reason to make the update. |
4 participants
Summary
Bumps
kotlinx.collections.immutablefrom 0.4.0 to 0.5.0-beta01.0.5.0-beta01 is binary-compatible with 0.4.0 — per KEEP-0459, the copy-returning methods on
Persistent*types were renamed (e.g.add→adding,remove→removing), and the old names remain as@Deprecated(WARNING)overloads. So this PR is purely a version bump and is reversible.Why no source changes?
bitwarden-android uses
kotlinx.collections.immutableas a read-only consumer —persistentListOf(...)etc. as factories, with values typed everywhere asImmutableList<T>/ImmutableMap<K, V>/ImmutableSet<T>(the read-only interfaces). None of the renamed methods onPersistent*are visible at any call site, so the Kotlin compiler emits zerokotlinx.collections.immutabledeprecation warnings after the bump.Changes
gradle/libs.versions.toml:kotlinxCollectionsImmutable0.4.0→0.5.0-beta01.MIGRATION_REPORT.md: per-phase audit of the migration (happy to drop this if you'd prefer to keep the repo root tidy).References
Test plan
:app(Standard + Fdroid debug, main + UnitTest),:authenticator(debug + UnitTest),:ui,:core(main + UnitTest),:testharness—BUILD SUCCESSFUL, zerokotlinx.collections.immutabledeprecation warnings.:core:testDebugUnitTest :ui:testDebugUnitTest—BUILD SUCCESSFUL(includesPersistentListExtensionsTest).🤖 Generated with Claude Code