Skip to content

Commit 70fb873

Browse files
committed
format
1 parent 1d8510d commit 70fb873

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

documents/forGitBranch/git_branch_standards.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -295,19 +295,19 @@ featureブランチでの作業中に、developブランチが更新された場
295295

296296
プリリクエストの承認(Approve)をもらった後、マージはレビュアー/レビュイーのどちらが行うべきか議論になる場合がある。
297297

298-
| 観点 | レビュアー派 | レビュイー派閥 |
299-
|-------|------------------------------------------|------------------------------------------------|
300-
| 説明 | 開発者の責務が、developブランチにマージするまでという役割分担の場合に有効 | 各開発者がその機能のリリースについて責任を負うモデルの場合に有効 |
301-
| 生産性 | ⚠️レビュアーがブロッキングになりがち | ✅️高い。コメントはあるがApproveしたので、適時対応してマージして、といった運用が可能 |
302-
| 統制 | ✅️レビュアーが管理しやすい | ✅️メンバーの自主性に依存 |
303-
| 要求スキル | ✅️低い。中央で統制を書けやすい | ⚠️開発メンバーの練度が求められる |
298+
| 観点 | レビュアー派 | レビュイー派閥 |
299+
| ---------- | ------------------------------------------------------------------------- | ------------------------------------------------------------------------------------ |
300+
| 説明 | 開発者の責務が、developブランチにマージするまでという役割分担の場合に有効 | 各開発者がその機能のリリースについて責任を負うモデルの場合に有効 |
301+
| 生産性 | ⚠️レビュアーがブロッキングになりがち | ✅️高い。コメントはあるがApproveしたので、適時対応してマージして、といった運用が可能 |
302+
| 統制 | ✅️レビュアーが管理しやすい | ✅️メンバーの自主性に依存 |
303+
| 要求スキル | ✅️低い。中央で統制を書けやすい | ⚠️開発メンバーの練度が求められる |
304304

305305
上記にあるように、そのプルリクエストで実装した機能を、本番環境にデリバリーする責務をどちらに持たせるかという観点で、意思決定することが多い。
306306

307307
本規約の推奨は以下。
308308

309-
* プロダクトオーナー(業務側)などでリリースタイミングを完全にコントロールしたいといった分業制を取る場合は、レビュアーがマージする
310-
* 各開発者により自律性を持たせ、アジャイル的に生産性を重視するのであれば、レビュイーがマージする
309+
- プロダクトオーナー(業務側)などでリリースタイミングを完全にコントロールしたいといった分業制を取る場合は、レビュアーがマージする
310+
- 各開発者により自律性を持たせ、アジャイル的に生産性を重視するのであれば、レビュイーがマージする
311311

312312
## 3. 永続ブランチ間で変更を取り込む
313313

0 commit comments

Comments
 (0)