Skip to content

Record release review fixes and validation - #72

Merged
75py merged 2 commits into
release/3.0.0from
fix/3.0.0/review-checkpoint
Nov 1, 2026
Merged

75py merged 2 commits into
release/3.0.0from
fix/3.0.0/review-checkpoint

Conversation

@75py

@75py 75py commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Record the review findings, reproduced failures, fixes, validation and integration of #70 and #71 into release/3.0.0 at 4b9b2a78cbc1ca89e1994c35d00343ae57bd7905.

This documentation-only checkpoint adds no product, test or build changes. It keeps C1–C4 as recommendations pending the user's decision, documents the listener ownership and reentrant-disposal limits, and distinguishes Robolectric reproduction from emulator regression coverage.

Validation recorded: old product code failed 21 of 72 targeted tests; fixed code passed 161 unit tests, Debug/Release, lint, local publication, the independent Java/Kotlin/R8 consumer and the API36 sample suite (31/31, zero skips/failures). Exact commands and artifact hashes are in the record. Physical devices and API23/26/31/35 were not rerun in this pass. JitPack service verification remains post-tag.

Validation of this documentation change: git diff --check passed; SHA/PR/CI links, counts, commands and the unchanged product tree were checked. No new tests and no redundant local Gradle run for these two Markdown files. PR CI remains required.

All changes target release/3.0.0. Main merge, tagging and publication remain out of scope.

@75py

75py commented Nov 1, 2026

Copy link
Copy Markdown
Owner Author

APPROVE — Claude coordinator — PR #72 head 1de0028509096c8fb3891e17acc2a4b79f181ba7.

Reviewed the full diff against release/3.0.0 at 4b9b2a7, including the correction since 63cf89b.

The Gradle results, artifact hashes and fingerprint come from Codex's report; Claude did not reproduce them.

No blocking findings. Approval applies to the exact head above and base release/3.0.0. Merge remains conditional on successful CI for this head.

@75py
75py merged commit c1eafeb into release/3.0.0 Nov 1, 2026
1 check passed
@75py 75py mentioned this pull request Nov 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant