pull requestのワークフロー
1265 ワード
code reviewの場合、Pull requestを使うのは良い方法です.
reviewerの人として、あなたが修正した場所がよく見えます.
では、このpull request全体の流れは何ですか?
新しいブランチごとに新しい機能やバグがあることを覚えておいてください.
1.
あるいは
2.コードの修正を開始し、
git add
など
3.現在のLocal branch pushをremote branchへ
git push -u origin feature-WEB-123-merged-branch
4.pull requestを起動すると、code reviewerの人にメールで通知されます
5.Code reviewerの人はあなたにいくつかのfeedbackをあげます.これらのfeedbackに基づいてコードを変更し続けます.
6.コードが監査を通過した場合、
merged-branchは最新のものがあると推定しているので、現在のbranchと衝突する可能性があります.
もちろんあなたも
すべてのcommitを変更してcommitにします
例えば「Fixed#WEB-123」
7.最後に、現在のbranch mergedを必要なbranchに渡す必要があります.
8.最後の最後は、feature branchまたはbug branchをローカルおよびリモートで削除することです.
reviewerの人として、あなたが修正した場所がよく見えます.
では、このpull request全体の流れは何ですか?
新しいブランチごとに新しい機能やバグがあることを覚えておいてください.
1.
git checkout -b feature-WEB-123-merged-branch origin/merged-branch
あるいは
git checkout -b bugfix-WEB-123-merged-branch origin/merged-branch
2.コードの修正を開始し、
git add
git commit
など
3.現在のLocal branch pushをremote branchへ
git push -u origin feature-WEB-123-merged-branch
4.pull requestを起動すると、code reviewerの人にメールで通知されます
5.Code reviewerの人はあなたにいくつかのfeedbackをあげます.これらのfeedbackに基づいてコードを変更し続けます.
6.コードが監査を通過した場合、
git rebase merged-branch
merged-branchは最新のものがあると推定しているので、現在のbranchと衝突する可能性があります.
もちろんあなたも
git rebase -i
すべてのcommitを変更してcommitにします
例えば「Fixed#WEB-123」
7.最後に、現在のbranch mergedを必要なbranchに渡す必要があります.
git checkout merged-branch
git merge --no-ff feature-WEB-123-merged-branch
8.最後の最後は、feature branchまたはbug branchをローカルおよびリモートで削除することです.
git branch -D feature-WEB-123-merged-branch
git push origin:feature-WEB-123-merged-branch