Tip:
Highlight text to annotate it
X
在這個視頻中,我想談談檢討其他人的代碼。
所以,當有人要你去檢查他們的代碼,你會收到一封電子郵件。
而該電子郵件會是這個樣子。
會被要求檢討標題,錯誤的主題,然後它會只給你審查鏈接,並鏈接到錯誤。
所以,如果錯誤就在這裡,我向下滾動。
你可以看到,我已經審查了2個補丁,但我只是要點擊回顧給你看。
因此,這是界面的樣子,你有概述就在這裡。
你有評論框就在這裡。
它會首先被您的電子郵件設置為一個問號。
如果你認同它,你只按+,如果你不同意它,你會打 - 。
然後你向下滾動,你會按publish。
現在,在你真正按+,你可能想要做檢討。
你要點擊每個文去做到這一點件,而你只是滾動通過它了,然後你看看是否一切才有意義。
現在在你看代碼之前,你應該問自己,如果,你被要求審查的任務實際上是我們希望在我們的代碼庫的東西。
因為它可能是一個社區成員,甚至是僱員,將嘗試添加覺得很重要的代碼,但真的不應該在那裡。
一旦你確定這是一個好任務,那麼你要代碼。
而剛剛更多的是討論對記得審查代碼,它不是一個判斷。
還你不會知道的一切。
所以,如果我要經過這裡,我真的不知道這是什麼意思,我可以只需雙擊這裡,我可以說:
為什麼要進行這種變化?這是真是沒有道理。
然後,你審查代碼的人會再回應你,你可以來回討論和對話。
你通常還要將代碼拉下到你的本地庫。
你並不總是有時間了,但它通常是一個好主意
然後你可以自己編譯並測試它。
當您完成檢閱某個文件,你可以點擊這個審查按鈕。
而這實際上並不需要。它只是幫助你,因為有時一個補丁中有20-30個文件,因此它變得混亂不知道哪一個有審查。
所以實際上盒子並沒有做任何事情,除了幫你。
所以24小時,在一天之內你通常想回到的人。
如果你無法查看一天內的補丁,你要告訴他們,你將無法審查補丁一個星期,或者其他幾天。
或者只是給他們一個開頭。
而這基本上只是給了他們一個機會去找不同的審稿人。
你也可能並不總是最好的被審查人。
因此,我並不是這裡的最好的人,我只想把那作為一個問號,投入別人的郵件。
比方說 rstrong,不同的人是對這次審查更好。
我只是把一個問號到他的電子郵件,說:
Robert 將處理這個修補程序審查得更好。
然後只需點擊提交。
而這只是轉發請求給他。
回顧這個補丁,我只是要把它明確地表示出來。
還有你想尋找出一些東西。
一個是編碼風格,確保它與文件的其餘部分保持一致,並確保它與Mozilla的標準相一致。
盡量不要在你會做什麼基礎上審查。
如果他們怎麼做是一樣好。
並總是試圖要有禮貌。
所以感謝那些提交補丁的人。
很多時候,他們都是貢獻者和他們沒有被支付薪水的補丁。
你想尋找出其他的事情都是安全問題,隱私,洩漏,確保它不會影響性能。
特別是如果它會影響啟動性能。
很多時候,你要問測試者。
有時,實際上他們是在不斷變化並不是真的可測試。
因此,在這些情況下,你並不需要。
對於大多數前後的代碼通常是可測試的
所以,你應該總是要求對人的測試。
你並不總是堅持審查,但確定的是有時堅持審查要求的人要寫一個測試。
尤其是如果是新功能,甚至是錯誤。
所以,我介紹了很多東西,但一個好主意是去尋找在MDN上的Reviewers Checklist
Reviewer Checklist MDN.並且它在第一個列表。
這將進入更多的細節。
它會涵蓋了很多我已經講過的東西,但會更深入。
這就是我想要在該視頻所闡述的˙。