「作業単位を見てブランチ切り直し」を判断するとは?
「作業単位を見てブランチ切り直し」を判断するとは?
一言でいうと
「この変更、1つのPRでまとめていい?」を判断する力
まず前提(超重要)
ブランチ = 作業のまとまり
PR = 承認単位
👉
承認できないものを一緒にしない
切り直すべき典型パターン(ディレクター判断)
① 変更理由が2つ以上ある
❌ 悪い例(1ブランチ)
-
テキスト修正(クライアント指示)
-
CSS微調整(ついで)
👉 理由が違う = 承認も別
✅ 正解
-
feature/text-fix -
feature/css-adjust
② マージタイミングが違う
❌ 悪い例
-
今日入れたい修正
-
来週リリース予定の対応
👉 まとめると判断できない
✅ 正解
-
今すぐ用
-
後日用
③ 影響範囲が違う
❌ 悪い例
-
トップページ文言
-
共通CSS
👉 共通CSSは影響大
✅ 正解
-
ページ単位
-
共通系は別ブランチ
④ 「戻す可能性」が違う
❌ 悪い例
-
A/Bテスト用修正
-
確定仕様の修正
👉 片方だけ戻せない
✅ 正解
-
戻す前提のものは別
⑤ PR説明が長くなる
❌ 危険サイン
-
PR説明が3段落以上
-
「ついでに」「あと」「その他」
👉 もう分けるべき
切り直し判断の質問テンプレ(自分に聞く)
PRを見るとき、これを自問👇
-
理由は1つ?
-
今マージしていい?
-
影響範囲は同じ?
-
一部だけ戻せる?
👉
1つでも「No」なら 切り直し
作業者への正しい言い方(超重要)
❌ NG
「ブランチ切り直して」
✅ OK(理由付き)
「変更理由が2つあるので、
文言修正とCSS調整でPR分けてもらえますか?」
👉
人格否定ゼロ・判断理由明確
実務でよくあるシーン(超リアル)
ケース
-
PR内容:
-
トップ文言修正
-
フッター色変更
-
不要タグ削除
-
ディレクター判断
「文言修正だけ今日入れたい。
他は別PRにしてください」
👉
ブランチ切り直し
レベル3の核心
-
コードを見てない
-
でも PRの良し悪しが分かる
-
事故を未然に防ぐ
👉
これが 「現場を回せる」判断力
コメント
コメントを投稿