Skip to content

ci: イメージ差し替えの PR を squash でマージする - #140

Merged
5ym merged 1 commit into
mainfrom
fix-bump-merge
Sep 16, 2026
Merged

5ym merged 1 commit into
mainfrom
fix-bump-merge

Conversation

@5ym

@5ym 5ym commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

リポジトリの設定が squash のみなので gh pr merge --merge が Merge commits are not allowed on this repository で落ちていた。イメージは焼けているのに deploy/ が更新されず、ワークフローだけ赤くなる。2026-09-12 と 09-16 の 2 回発生していて、どちらも手でマージして復旧した。

🤖 Generated with Claude Code

https://claude.ai/code/session_01JgGYf5qbjDXYhtzcpLsGvv

リポジトリの設定が squash のみなので、gh pr merge --merge は
"Merge commits are not allowed on this repository" で落ちていた。
イメージは焼けているのに deploy/ が更新されず、ワークフローだけ赤くなる
(2026-09-12 と 09-16 の 2 回発生。どちらも手でマージして復旧した)。

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JgGYf5qbjDXYhtzcpLsGvv
@claude

claude Bot commented Sep 16, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 46s —— View job


レビュー結果

変更内容: .github/workflows/docker-publish.yml の自動マージ処理を --merge (マージコミット) から --squash に変更。リポジトリの設定が squash のみ許可のため、従来の --merge は Merge commits are not allowed on this repository で失敗し、イメージは焼けているのに deploy/ の更新 PR だけが宙に浮く不具合が起きていた、という PR 本文の説明と実際の症状は整合しています。

確認した観点

  1. 不具合: --merge → --squash への変更のみで、フォールバックの --auto --squash も対応する形に揃っており、ロジックの誤りは見当たりません。squash マージでも deploy/deployment.yaml のみを変更するコミットになるため、paths-ignore: deploy/** によるループ防止の仕組み (L6-9) も従来通り機能します。
  2. セキュリティ: 権限・シークレットの扱いに変更なし。問題ありません。
  3. 設計: 既存のコメントスタイル (該当箇所の直前にある「なぜこうしているか」を説明する行) に沿った1行コメントが追加されており、一貫性があります。差分も最小限で過不足ありません。
  4. テスト: GitHub Actions ワークフローの変更であり、既存もテスト対象になっていないため、テスト追加は不要と判断します。

問題点は見つかりませんでした。原因(squash-only 設定と --merge の不整合) の特定と修正が的確で、実際に2回インシデントが起きている再発防止策として妥当な変更だと思います。

@5ym
5ym merged commit 9f85c9f into main Sep 16, 2026
2 checks passed
@5ym
5ym deleted the fix-bump-merge branch September 16, 2026 08:18
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