Skip to content

chore: record Android 3.1.1 release - #1904

Open
piotr-iohk wants to merge 1 commit into
masterfrom
codex/record-release-3.1.1
Open

piotr-iohk wants to merge 1 commit into
masterfrom
codex/record-release-3.1.1

Conversation

@piotr-iohk

Copy link
Copy Markdown
Collaborator

Description

Record the shipped Bitkit 3.1.1 build 197 on master. Copy the published changelog and remove the fragments already included in it so the next release does not repeat those entries. Published 3.1.1 release · Release PR

Design

N/A — no UI changes.

Preview

N/A

QA Notes

Compared the Gradle version fields and changelog with the published v3.1.1 tag. git diff --check passed. Mainnet QA covers the shipped tag. 3.1.1 QA results

Models used

  • Planning/scoping: GPT-6 (reasoning: Not exposed)
  • Implementation: GPT-6 (reasoning: Not exposed)
  • Review: Not performed

The session exposes the model family. The reasoning setting is unavailable.

@piotr-iohk
piotr-iohk requested review from a team, ben-kaufman and ovitrif and removed request for a team November 12, 2026 19:02
@greptile-apps

greptile-apps Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] This release-recording change appears safe to merge.

Summary

Records the shipped Android 3.1.1 build 197 on master.

  • Bitkit’s project records now identify the shipped 3.1.1 release.

No actionable issues found.

Reviews (1) · Last reviewed commit: "chore: record Android 3.1.1 release" · Reviewed by Greptile

@talosmachina talosmachina left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No findings. Records the shipped 3.1.1 (build 197) on master: version fields, the 3.1.1 changelog section and compare links, and removal of the four fragments it consumed. Reviewed 0443ae5, full tier, static (Bitkit Android branches are not built here; reasoned from the diff, the v3.1.1 tag and CI).

What I checked, and 3 candidates I ruled out

Against the tag: CHANGELOG.md on the PR head is byte-identical to CHANGELOG.md at v3.1.1; versionCode = 197 and versionName = "3.1.1" match the tag's app/build.gradle.kts.
Base: the merge base is the current master tip (db3b1b2), so nothing on master is overwritten.

Ruled out

  • A consumed fragment left behind, so 3.1.1 entries repeat in the next release: the four entries with fragments on master (#1798, #1801, #1803, #1870) are the four deleted; #1824 and #1873 never had a fragment on master. A keyword pass over the 26 remaining fragments matched none of the 3.1.1 entries (the two "Lightning node" hits, #1536 and #1756, are different fixes).
  • An unreleased fragment deleted by mistake: only the four above are removed.
  • Compare links: [Unreleased] now starts at v3.1.1 and [3.1.1] spans v3.1.0...v3.1.1, consistent with the existing entries.

CI: Greptile passed; build-and-test, lint, build-local and compose-ui were pending when this was posted.

Merge confidence: 4/5, the change is version fields and changelog only, but CI was still running at review time.

@github-actions

Copy link
Copy Markdown
Contributor

Regtest APK

Built from 0443ae5 (run).

Download bitkit-dev-debug universal APK (expires in 30 days).

@ovi-reviewer ovi-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: 💬 Comment. The release record matches published v3.1.1; current-head Compose UI CI is failing.

Findings: None.

QA for 0443ae5:

Check: Match the published release ✅ Static comparison confirms version 3.1.1, build 197, and the published changelog.

CHANGELOG.md is byte-identical to the published tag, which points to 0f23fc2. The Gradle version fields also match that tag.

Check: Preserve unreleased changelog entries ✅ Static inspection confirms the four deleted fragments are recorded in 3.1.1 and the remaining entries are preserved.

Each of the four deleted fragments appears verbatim in the published changelog. The 26 remaining fragments describe other changes. The release comparison links are correct, and git diff --check passes for db3b1b2..0443ae5.

Check: Current-head CI ⚠️ Builds, unit tests, lint and local E2E pass, but Compose UI has eight failures.

The Compose UI job reports six window-focus failures and two Compose timeouts. Its failure also fails the aggregate build check; the build-and-test job itself passed. Lint and local E2E also passed. I have not reproduced the Compose failures against the base or established that this PR causes them.

No app build or device cases were run for this review: the QA notes list static checks, and this diff only records release metadata and consumes changelog fragments. The linked release QA spreadsheet required authentication, so I did not use it as evidence. The earlier review at this head agrees with the independently checked release-record comparisons.

Full review covered all six changed files in db3b1b2..0443ae5. No runtime or test sources changed.

Twin: bitkit-ios#1319 chore: record iOS 3.1.1 release matches this release-record scope, using iOS build 204 and its platform-specific changelog entries.


Reviewed by gpt-6.1-sol-high

This branch has not been deployed

No deployments
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.

2 participants