Release v1.2.0 and re-pin keep to b06b61f - #471
Conversation
|
Warning Review limit reached
Next review available in: 43 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (10)
WalkthroughThe Android app version changed from 1.1.9 (code 27) to 1.2.0 (code 28). The pinned version identifier in ChangesRelease update
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Prepares v1.2.0:
versionCode27 to 28,versionName1.1.9 to 1.2.0, and re-pinskeep.versionfrom88dd3bctob06b61f.Why the re-pin matters here
keep.versiondecides which keep core gets built into the app, and it had fallen four commits behind keep's main. One of those is load-bearing for this release:95e7626(#932)keep-core/src/entropy.rse98cea6(#933)keep-clionly9cfe24a(#931)b06b61f(#934)#470 landed the Android side of the degraded-RNG work, but the pinned core still predated the core-side fix in
keep-core/src/entropy.rs. Releasing on the old pin would have shipped one half of that change without the other. The re-pin is what makes the release coherent, not housekeeping alongside it.Release contents
Five commits since v1.1.9:
Minor rather than patch: #468 changes how stored values are bound to their key, and #469 changes share-deletion ordering.
Test plan
verifyKeepVersionpasses againstb06b61fkeep.versionis a 40-char lowercase hex SHA that exists on keep's main, matching the format gate inbuild.gradle.ktskeep.versionpreserved, so the diff is the SHA alonebuild.gradle.ktsrequires JDK 21 for any assemble/compile/lint task and this machine has only JDK 17, so no localassembleDebugwas run. The APK build, lint, locale-drift and hardcoded-string gates all need to pass here before this is merged or tagged.verifyKeepVersionwas run against a throwaway clone viaKEEP_REPOrather than the localkeepsymlink. The task's own failure message suggestsreset --hardplusclean -fdxon that path, which resolves to the real keep repository and would delete untracked work there.Summary by CodeRabbit