Repository navigation
chore: update paykit to rc70 - #1435
Conversation
Regtest APKDownload bitkit-dev-debug universal APK (expires in 30 days). |
|
There was a problem hiding this comment.
Verdict: ✅ Approve
Review: diff 2 files.
The iOS pair PR synonymdev/bitkit-ios#886 updates to the same rc70 release and documents the same Homeserver requirement and recovery limits.
QA:
No tests ran because this PR declares no user-visible change and lists journeys and manual tests as N/A.
Reviewed by gpt-6.1-sol-high via gh-pr-review-loop skill
Commands: @ovi-reviewer review · test · retest
This PR upgrades the Android Paykit dependency to rc70 and documents the shared-state deployment requirement.
Follows merged #1401; targets
master.Description
com.synonym:paykit-androidto0.1.0-rc70ingradle/libs.versions.toml; GitHub Packages resolution and the Maven-local exclusion remain unchanged.docs/pubky.mdthat shared-state deployments require Homeserver 0.15 or newer on every serving instance. Paykit does not upgrade or verify the homeserver; pending markers and the five-minute uncertain-write cooldown remain required.No consumer runtime, test, or UI changes.
Design
N/A — no UI changes.
Preview
N/A
QA Notes
Device journeys and performance were not tested; no performance improvement is claimed.
Journeys
N/A — no user-visible behaviour change.
Manual Tests
N/A
Automated Checks
devDebugUnitTestRuntimeClasspathwithdependencyInsight.--rerun --no-build-cache -Pkotlin.incremental=false— all 3,587 tests passed with no failures, errors, or skips; all 190 source compiler warnings match the baseline.ignoreFailures— all 19 baseline Detekt warnings, including one formatting finding, and the single Gradle plugin deprecation are unchanged; no new findings.