Re: [討論] 就算提早做完是不是不要回報比較好

loadingN04/20 07:35上版的功能切乾淨，是本來就該做的，果然過太爽

CoNsTaR04/20 07:40真的，每次看到那種一個 PR 上千行的看都不用看就知道一

CoNsTaR04/20 07:40定菜鳥發的

vi00024604/20 08:43一個pizza切兩塊跟切十塊的概念 不過做的事還是一樣多

yamagishi04/20 08:52code review 可以比較簡單是真的，一次丟太多很難面面

yamagishi04/20 08:52俱到

naestnecniv04/20 09:38但切完PR，review的速度比我PR發起的速度還慢就很麻

naestnecniv04/20 09:38煩了。

Galbygene04/20 13:14請問PR是什麼的縮寫

rabbitu0404/20 13:18pull request

Galbygene04/20 14:59謝謝

s06yji304/20 15:05Review通常都超拖的啊

orangelite04/20 18:52有點好奇原po是寫前端還是後端？

orangelite04/20 18:52我自己前端的經驗一個頁面就是一個 pr

orangelite04/20 18:52分很多 pr 畫面不完全感覺很怪…

foreverk04/20 19:15你的畫面如果一頁有10個新的元件，你發在一個pr內的話

foreverk04/20 19:15，review的人要嘛花很常時間看，或是分好幾天看卡住你

foreverk04/20 19:15的pr，你切分好給人review同時你還能繼續開發或是改其

foreverk04/20 19:15他review的pr

s06yji304/20 19:18這樣怎知道10個PR merge起來沒有問題...

s06yji304/20 19:20不過通常一個元件就可以是一個PR了

foreverk04/20 19:25也不用10個pr，關聯性較高的合起來發三四個，都比一整

foreverk04/20 19:25包出去好吧

s06yji304/20 19:27啊，應該說該break down的應該是task而不是PR

safe04/20 19:44適用場景：案子時程不趕、團隊 junior 偏多

Ekmund04/20 23:49理想狀態是這樣 如果不是臨時改義大利麵這週五上的話

holmes213604/21 14:36切得好還可以避免conflict 的機會

holmes213604/21 14:37Review的人也輕鬆多

d006826704/21 16:06好聞

bobokeke04/21 23:33我自己習慣一個PR不多過4個檔案或250-300行程式，除非是

bobokeke04/21 23:33asset/config/generated files

bobokeke04/21 23:34不過我常常一天連發10張PR，哈哈哈

bobokeke04/21 23:34切PR是一門藝術