今年年初我朋友面試進到一間港商，是一家電商小公司，最近跟他吃飯在聊公司的開發流
程
聊著聊著，竟然發現他們有使用 Github 但卻沒有發 PR
流程大概就是
切 branch -> 開發 -> 做完丟 branch name 給上頭 review
我一聽就覺得超怪，我朋友一開始進去也有問其他同事，但他們就是一臉很正常的樣子，
他之後也習以為常了
有用 Github 但不發 PR 的公司真的是第一次聽到...
--
[討論] 不發 PR 的公司會很怪嗎
軟工
85404
今年年初我朋友面試進到一間港商，是一家電商小公司，最近跟他吃飯在聊公司的開發流
程
聊著聊著，竟然發現他們有使用 Github 但卻沒有發 PR
流程大概就是
切 branch -> 開發 -> 做完丟 branch name 給上頭 review
我一聽就覺得超怪，我朋友一開始進去也有問其他同事，但他們就是一臉很正常的樣子，
他之後也習以為常了
有用 Github 但不發 PR 的公司真的是第一次聽到...
--
→ sisdad1樓有沒有一種可能是你見過的世面太少 08/16 13:06
→ surfingbboy2樓不會 08/16 13:11
t199608043樓
08/16 13:15
08/16 13:15→ stepnight4樓就是用不用這功能而已，做得也沒哪裡不一樣 08/16 13:15
→ stepnight5樓還是你覺得用什麼需求一定要用到PR才能做到 08/16 13:15
比如說可以直接在你的 code 上去做評論
沒用的話就會變成是「在 xx function 內的某個 function...」，溝通起來就會沒那麼
直觀
→ Newtype6樓至少有review了 08/16 13:15
t199608047樓是覺得有發pr比較正式吧 08/16 13:18
→ qoo19918樓Linus 也沒用PR 該怎麼辦 08/16 13:20
mercurycgt689樓trunk based development: 08/16 13:26
我們公司就是用 trunk based，每次 merge 回主線就是發 PR
→ bear141410樓方法(any)是靈活的 本質(review)才是重要的 08/16 13:27
abc092200111樓肯定有段故事的 08/16 13:29
Imin090512樓有review就不錯了吧… 08/16 13:32
feathergod13樓至少有review 前公司不review還會在production branc 08/16 13:35
→ feathergod14樓h開發 08/16 13:35
MoonCode15樓沒有特別說明為何這樣做的話 就是雷 08/16 13:37
NDark16樓PR只是一種merge的備忘錄，只要事情沒有多到記不住。merge 08/16 13:38
→ NDark17樓也可以達到相同功能。當然搭配自動測試這是兩件事。 08/16 13:38
lwecloud18樓小公司有啥好意外 功能做出來賣錢才是重點 08/16 13:40
NDark19樓這就像是一人開發要不要用issue tracking 08/16 13:49
→ Obama1920樓發pr有法律規定嗎？ 08/16 13:50
→ NDark21樓如果事情沒有多到記不住自己不用裝模作樣開issue給自己 08/16 13:50
我比較疑惑的點是明明有 PR 這個功能可以方便 code review，反而要用其他繞路的方式
來進行，覺得很奇怪
NDark22樓對於更直接當面討論的團隊來說，說不定PR才是繞路。 08/16 13:52
→ answermangtr23樓只是流程不一樣而已 還是有review 08/16 13:54
但有發 PR 這流程會讓 review 變得輕鬆點
→ shooter55524樓沒有review就不用PR MR啦 08/16 14:02
→ nh60211as25樓但有發 PR 這流程會讓 review 變得輕鬆點 => 不一定 08/16 14:03
NDark26樓同樓上 08/16 14:09
abccbaandy27樓有真review就屌打大部分公司了... 08/16 14:36
→ ssccg28樓看起來只是你習慣用github的UI而已 08/16 14:47
→ bheegrl29樓大家有默契就好了 08/16 15:04
chopinmozart30樓Real man test on production 08/16 15:40