From 7d5dcd454e439a3e64a8a851ca4f00f4d0f639b9 Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Mon, 26 Oct 2026 11:20:17 -0400 Subject: [PATCH] docs: two comments state the rule rather than its history (#6277) Co-Authored-By: Claude Opus 5.5 --- .../com/fabledsword/minstrel/update/data/UpdateRepository.kt | 2 +- internal/server/release_version_test.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/android/app/src/main/java/com/fabledsword/minstrel/update/data/UpdateRepository.kt b/android/app/src/main/java/com/fabledsword/minstrel/update/data/UpdateRepository.kt index b66d91ff..5b1e96a9 100644 --- a/android/app/src/main/java/com/fabledsword/minstrel/update/data/UpdateRepository.kt +++ b/android/app/src/main/java/com/fabledsword/minstrel/update/data/UpdateRepository.kt @@ -62,7 +62,7 @@ class UpdateRepository( private val installedCode: Long, private val installedName: String, private val offers: Boolean, - // Where the updater before this one kept its download, never cleaned up. + // A download left by older builds of the app, which nothing else removes. private val legacyApk: File? = null, ) { private val turns = Mutex() diff --git a/internal/server/release_version_test.go b/internal/server/release_version_test.go index 5967cf30..ac95415d 100644 --- a/internal/server/release_version_test.go +++ b/internal/server/release_version_test.go @@ -578,8 +578,8 @@ func TestVersionName_AndroidSourcesCount(t *testing.T) { // The app offers an update whenever the server's APK has a different name // from its own, so the APK's name must move for Android commits and ONLY for -// them. Named from the whole repo, every server-only deploy offered each -// phone a rebuild of the app it already had. +// them. A name taken from the whole repo would offer every phone a rebuild of +// the app it already has on each server-only deploy. func TestApkName_MovesOnlyForAndroidCommits(t *testing.T) { const ( app = "1757443736" // 2025.09.09.1848