只有完美代碼不夠的，如何做一個完美的Pull Request？
想要提高團隊績效，找到瓶頸是第一步?，F實中，最大的限制因素不是編碼速度，而是代碼審查。因此，為了加快審查速度，筆者對比了兩種pull request：
- 注釋很少并且快速合并的pullrequest
- 有很多注釋，需要多輪審查的pull request
我的結論是，有九種方式能讓審查pullrequest更輕松。
1.添加關于“為什么”的代碼注釋
在寫一個新功能的時候，會有很多與之相關的信息。寫代碼時要全盤考慮需求，第三方系統(tǒng)的局限性，以及和遺留代碼庫的交互。但是別人不了解其上下文來源，所以看到這個代碼時會問“它為什么在這?”或是“為什么要選擇這種方法?”
因此要通過添加解釋性的注釋，讓閱讀代碼的人提前知曉“為什么”。筆者不認同一些人宣揚的觀點：注釋有害，應當忽略。
注釋有很多種類。那些描述代碼用途的確實是累贅。提取一個方法，采用一個精心挑選的命名，就能消除這種麻煩。另一方面，當解釋為什么這樣寫代碼時，也增加了代碼閱讀者的信息量。這些注釋將閱讀者的認知水平理想化地提高到了與編碼人員相同的層級，這有助于增進對代碼的理解。
筆者的注釋通常會給出類存在的原因、相關資源的鏈接以及代碼的前因后果：
- # First Crew Dragon launch was postponeddue to bad weather,
- # and now we needan event for the "second" first launch.
- # Hence the stupidname.
- classSecondFirstCrewDragonLaunch
- ...
- End
2.描述清晰
有關pull request的描述為審查者提供任務最初的上下文，包括：
- 標簽的鏈接。
- 對已完成事件的總結(如果不能從pullrequest的標題中看出)。
- 相關pull request的鏈接(例如在另一服務中的相關變化)。
不要把自認為理解代碼需要的信息放在對pullrequest的描述里，應當進行代碼注釋：它們的效果更加顯著，有助于未來代碼閱讀者的閱讀。
3.精簡pull request
這是一項強大的技術，谷歌甚至就小型pullrequest的益處單獨寫了一篇文章(https://google.github.io/eng-practices/review/developer/small-cls.html)。以下是筆者最喜歡的小型pull request的特點：
- 審查更徹底
- 審查更快捷
- 更易合并(頻繁的合并能減少沖突)
- 如果被拒絕，浪費的精力更少
以下辦法能使小型pull request的編寫更簡單：
- 將重構提取到單獨的pull request中
- 將大型功能分解(即使它們不是面向用戶的)
- 學一些git小竅門很有幫助，把git add --patch和git rebase --interactive當成朋友
- 把長期運行的功能分支設置為pull request的目標，而非master的目標：
4.快速回應審查
處理審查注釋通常比較費時，需要修復打字錯誤、添加遺漏的測試案例、對方法重命名等。如果你能快速完成，你的同伴就能花更少的時間來記憶與pull request相關的信息。
但這種方法的缺點是會增加上下文切換的工作量，替代方法是使用番茄工作法(Pomodoro technique)：每工作25分鐘穿插一次短暫的休息。它能讓人更專注、更有成效、更健康，并減輕疲勞度，休息后的上下文切換也會進行得更加自然。負面的破壞性影響雖然沒有消失，但是會大大降低。
5.給自己的pull request注釋
為某些變化(例如刪除和重構)添加解釋性的代碼注釋是沒有意義的，應當考慮為自己的pull request注釋，給審查者提供更多的上下文。
6.在創(chuàng)建pull request前重定新master的基準
這樣做有很多好處：
- 測試可能在本地分支中會通過，但在應用的最近更新時會失敗。
- 能夠使用最新添加的功能(例如新的工具類)。
- 審查者如果沒能找到近期的變化，就會感到困惑。
相比合并，筆者更喜歡重定基底，因為重定基底使得分支僅包含相關的提交。
7.不要修改經過審查的提交——要發(fā)送新的
要在單獨的提交中處理審查注釋，而不是修改或者除去更改。這樣能夠讓審查者更容易核對在上次審查后發(fā)生的變化：
8.在實現功能之前討論整體方法
這可以省下很多時間。在要處理更復雜的重構和功能之前，先與同事討論一下方法。與其他的開發(fā)人員討論，解釋這項任務和你的想法，他們也許會表示贊成，也許會提出更好的方法。
很多時候筆者都面臨著初步協(xié)調的缺失，好幾天的工作成果白白浪費。想象一下你連續(xù)五天做一件事情，結果卻聽到“對不起，其實我們不需要……”想要把自己從失望中拯救出來，你得盡早獲得反饋。
9.感謝審查者的建議
深刻地理解別人做出的改變并且提出有用的建議需要付出很多努力——請對于這一事實表達認同和感激。記住，代碼的變化是短暫的——與隊友的關系卻是永恒的。
減少在代碼審查上花費的時間，團隊表現會很快得到優(yōu)化，在下一次的pull request中運用這些小竅門，結果會大有不同。
