2014-07-10 22 views
2

當要素分支需要與我們的開發分支合併時,我們使用存儲並創建拉取請求。我們有一個標準,每個實現的功能都需要在開發分支中進行一次提交。我們實現這一目標的方式是通過處理特徵分支中的特徵,然後將所有提交壓縮到一個特徵中。此時,我們將功能分支推送到Stash並創建拉取請求。如何在存儲拉取請求中維護代碼審閱歷史

當拒絕請求並且代碼需要更改時,就會產生挑戰。發生這種情況時,我們對功能分支進行所有代碼更改,修改提交,強制將分支推送到Stash,並重新打開請求(我們不會創建新的請求)。 Stash顯示1個提交被刪除,1個被添加。這個添加的提交包含更新後的代碼。但是,我們在代碼審查中爲刪除的提交寫的所有評論都未顯示在添加的提交中。基本上,當我們查看添加的提交時,無法查看代碼的哪些部分是從先前的代碼審閱中更新的。

我的問題是,如果有人知道更好的方式在Stash中做代碼評論,並在Git中保持一個乾淨的提交歷史記錄。

謝謝!

回答

1

我們在團隊中使用相同的協議。唯一的區別是我們在推送提交時不會壓縮我們的提交。然而,擠壓/不擠壓並不能解決您的問題。

每當我們的代碼需要改變後,我們已經創建了我們的拉請求,我們不要拒絕拉請求,我們只是在審查中提到的細節。所以不是。一旦開發人員進行了必要的更改並將更改推送到其功能分支,則自動更新拉取請求,並始終在Git中維護歷史記錄以及拉取請求的歷史記錄。

+1

我們也開始嘗試不拒絕審查。但是,它似乎對代碼審查意見沒有影響。爲原始提交寫入的註釋不會在隨後的提交(包含修復的提交)中顯示。無論如何,我感謝你的幫助! – MaxK