Skip to content

cherry-pickリリース後の同期PRの版数コンフリクト解決手順をスキルに追記#6405

Merged
TinyKitten merged 1 commit into
devfrom
docs/release-sync-version-conflict
Jul 12, 2026
Merged

cherry-pickリリース後の同期PRの版数コンフリクト解決手順をスキルに追記#6405
TinyKitten merged 1 commit into
devfrom
docs/release-sync-version-conflict

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Jul 12, 2026

Copy link
Copy Markdown
Member

概要

リリースブランチを master から切って特定 PR だけ cherry-pick したリリース(create-release-pr の「この変更だけ」指定など)の後、dev<-master 同期 PR が版数ファイルで衝突する事象と、その解決方針・具体手順を AI 向けドキュメントとしてスキルに追記します。v10.9.1 リリース(#6402 → 同期 #6404)で実際に遭遇したケースの知見です。コード変更は含みません。

変更の種類

  • バグ修正
  • 新機能
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

  • sync-dev-from-master スキルに「版数ファイルのコンフリクト解決(cherry-pick / hotfix リリース後)」節を追加
    • 衝突するのは版数3ファイル(android/app/build.gradle / app.config.ts / ios/TrainLCD.xcodeproj/project.pbxproj)のみである点
    • 解決方針: semver はリリース版数(master)、ビルド番号は max(dev, master)(単調増加維持・canary 既発行分との衝突回避)
    • origin/dev をマージ →--ours で確定 → ビルド番号だけ引き上げ → semver 差分のみを確認してコミットする具体手順
    • 手順4に「衝突時は解決コミットを1つ積む」例外ノートを追記
  • finalize-release スキルの手順6に、マージ前のコンフリクト確認ノート(CONFLICTING なら Ruleset 緩和より先に版数解決)を追加

テスト

  • npm run lint が通ること
  • npm test が通ること
  • npm run typecheck が通ること

省略: ドキュメント(.claude/skills/**/SKILL.md)のみでコード変更が無いため、lint / test / typecheck は非該当。

関連Issue

スクリーンショット(任意)

Summary by CodeRabbit

  • ドキュメント
    • リリース手順を改善し、同期時に発生する競合を事前に確認できるようになりました。
    • バージョン情報の競合解決ルールと、リリース版・ビルド番号の確認手順を明確化しました。
    • 変更後にブランチの内容が正しく反映されていることを検証する手順を追加しました。
    • バージョン確定時に、手動での確認を必須とする運用を明記しました。

@TinyKitten TinyKitten self-assigned this Jul 12, 2026
@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 412e41cc-ba19-46b6-b902-e7e3d757f694

📥 Commits

Reviewing files that changed from the base of the PR and between b91a4b8 and 052f77d.

📒 Files selected for processing (2)
  • .claude/skills/finalize-release/SKILL.md
  • .claude/skills/sync-dev-from-master/SKILL.md

📝 Walkthrough

Walkthrough

同期ブランチで版数ファイルが衝突した場合の解決手順を追加し、同期PRのマージ前に競合状態とブランチの包含関係を確認するようリリース手順を更新します。

Changes

リリース同期と衝突解決

Layer / File(s) Summary
版数ファイル衝突の解決手順
.claude/skills/sync-dev-from-master/SKILL.md
版数ファイル衝突時の例外処理、semver・ビルド番号の採用ルール、ユーザー確認、解決コミット作成と検証手順を追加。
同期PRのマージ前検証
.claude/skills/finalize-release/SKILL.md
マージ前にPRの競合状態を確認し、必要に応じて版数ファイルを解決した後、ブランチの包含関係を検証してマージを続行する手順を追加。

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • TrainLCD/MobileApp#6297: 同じsync PRのマージ手順付近で、マージ前後の実行条件と検証を更新しています。

Suggested labels: ios, android

Poem

ぴょんと跳ねて、衝突を発見
版数そろえて、確認ぴょん
dev と master、仲良く包含
承認もらって、マージへ進む
うさぎも安心、リリース日!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/release-sync-version-conflict

Comment @coderabbitai help to get the list of available commands.

@TinyKitten

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@TinyKitten TinyKitten merged commit fca5fba into dev Jul 12, 2026
5 checks passed
@TinyKitten TinyKitten deleted the docs/release-sync-version-conflict branch July 12, 2026 21:01
@TinyKitten TinyKitten mentioned this pull request Jul 12, 2026
9 tasks
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