Repository navigation
chore(release): v1.2.27-dp2 - #7
Merged
Merged
Conversation
This was referenced Oct 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bump the fork release to
v1.2.27-dp2(.release-version+versionName). Merging triggers release.yml, which publishes a signed build that includes the DoubleCode rename (#6) and the GitHub client ID (repo variableAND_CODE_GITHUB_CLIENT_ID).versionCodestays 66 (same signing key, so dp1 -> dp2 installs over the top).Pre-PR review
判定: APPROVE
ブロッカー
なし
提案(非ブロッキング)
app/build.gradle.kts:128で versionCode を 66 のまま据え置く件について。同じ署名鍵なら同じ versionCode でも上書きインストールできます。Android が拒否するのは versionCode を下げる場合だけなので、理解のとおりです。ただし、注意点が 2 つあります。fdroid-repository.yml:26のガードで F-Droid リポジトリを公開していないので、現状は影響がありません。ただ、Obtainium などを使う利用者がいる場合は、更新通知が出ない可能性があります。ReleaseMetadataTest.kt:13の期待値も合わせて更新する運用にすると安全です。テストを変えると upstream との差分になるので、今回のように据え置く判断も合理的です。RELEASE_KEYSTORE_BASE64などの secret を変えていないこと)。secret を差し替えている場合は、署名の不一致でインストールが失敗します。gh pr merge)でマージしたときです。GITHUB_TOKENで動く Action からマージすると、その push イベントでは別のワークフローが起動しません。dp1 のリリースが同じ手順で起動したなら問題ありません。チェック済み項目
.release-versionがv1.2.27-dp2で、先頭のvを除くと1.2.27-dp2になります。app/build.gradle.kts:129のversionName = "1.2.27-dp2"と一致します。versionCode = 66もReleaseMetadataTest.kt:13の期待値と一致します。dp1/dp2/1.2.27を検索しました。dp1は残っていません。docs/RELEASE.mdに1.2.27の記述がありますが、upstream の F-Droid metadata のある時点のスナップショットについての説明なので、更新は不要です。push、branches: [main]、paths: ['.release-version', ...]の組み合わせなので、.release-versionを変えたコミットが main にマージされれば起動します。タグはrelease.yml:104で.release-versionからv1.2.27-dp2として読み出されます。gh release viewで既存のリリースが無いと判定されると、gh release create --target ${{ github.sha }}で新しいリリースとタグが作られます。dp1 とはタグ名が違うので衝突しません。release.yml:78でvars.AND_CODE_GITHUB_CLIENT_IDをGITHUB_CLIENT_IDとして渡し、app/build.gradle.ktsのgithubClientIdからBuildConfigに埋め込まれる経路を確認しました。github flavor の debug / release の両方に入ります。fdroid の APK ではenv -uで意図的に外されています。git diffは実行していません。報告どおり 2 か所が変わっていることを、ファイルの現在値で確認しました。i18n、Kotlin コード、テストに変更はありません。