作者:
SuKamo (Su_Kamo)
2024-08-16 12:53:01今年年初我朋友面試進到一間港商,是一家電商小公司,最近跟他吃飯在聊公司的開發流
程
聊著聊著,竟然發現他們有使用 Github 但卻沒有發 PR
流程大概就是
切 branch -> 開發 -> 做完丟 branch name 給上頭 review
我一聽就覺得超怪,我朋友一開始進去也有問其他同事,但他們就是一臉很正常的樣子,
他之後也習以為常了
有用 Github 但不發 PR 的公司真的是第一次聽到...
作者: sisdad (sisdad) 2024-08-16 13:06:00
有沒有一種可能是你見過的世面太少
就是用不用這功能而已,做得也沒哪裡不一樣還是你覺得用什麼需求一定要用到PR才能做到
作者:
Newtype (你快樂所以我快樂)
2024-08-16 13:15:00至少有review了
作者:
qoo1991 (笨哲)
2024-08-16 13:20:00Linus 也沒用PR 該怎麼辦
作者: mercurycgt68 (發芽的吉它手) 2024-08-16 13:26:00
trunk based development:
作者: bear1414 (story) 2024-08-16 13:27:00
方法(any)是靈活的 本質(review)才是重要的
作者:
Imin0905 (Imin0905)
2024-08-16 13:32:00有review就不錯了吧…
至少有review 前公司不review還會在production branch開發
作者:
MoonCode (MoonCode)
2024-08-16 13:37:00沒有特別說明為何這樣做的話 就是雷
作者:
NDark (溺於黑暗)
2024-08-16 13:38:00PR只是一種merge的備忘錄,只要事情沒有多到記不住。merge也可以達到相同功能。當然搭配自動測試這是兩件事。
作者:
lwecloud (CloudEX)
2024-08-16 13:40:00小公司有啥好意外 功能做出來賣錢才是重點
作者:
NDark (溺於黑暗)
2024-08-16 13:49:00這就像是一人開發要不要用issue tracking
作者:
Obama19 (^_^)
2024-08-16 13:50:00發pr有法律規定嗎?
作者:
NDark (溺於黑暗)
2024-08-16 13:50:00如果事情沒有多到記不住自己不用裝模作樣開issue給自己對於更直接當面討論的團隊來說,說不定PR才是繞路。
但有發 PR 這流程會讓 review 變得輕鬆點 => 不一定
作者:
NDark (溺於黑暗)
2024-08-16 14:09:00同樓上
作者:
ssccg (23)
2024-08-16 14:47:00看起來只是你習慣用github的UI而已
Real man test on production
作者:
luke72 (ccc)
2024-08-16 15:48:00團隊才幾個人發PR是能多賺錢嗎?repo搞不好是單人開發一堆新手看了廣告文,就想拿5000人團隊制度套到5人團隊
作者:
zxc8787 (摸斗哈壓庫)
2024-08-16 15:56:00git的具體使用流程應該是配合公司吧有pr就有pr,沒有也不會怎樣吧
有可能剛學會怎麼用git,沒時間也沒心力測試這個流程
作者:
wei115 (ㄎㄎ)
2024-08-16 16:25:00pr是github的功能吧?如果只是把github當git server沒pr也ok
作者:
jackflu (jackflu)
2024-08-16 16:36:00我覺得上面部份人其實不懂 PR,所以看不懂你的納悶,哈哈
pr都不懂別想說git flow自己管是多會管理拉..
小團隊還好吧 如果一個repo有上百個人在開發哪可能非用不行
作者: moom50302 (武林三羚鱷) 2024-08-16 18:00:00
內文加留言 滿滿的工程師相輕
有可能是在Review完由Reviewer Merge,那不一定要MR, PR
作者:
crazwade (crazwade)
2024-08-16 19:05:00公司就是 有分不同分支開發最後由主開發人來merge 不懂為什麼不用 PR就好
作者:
hegemon (hegemon)
2024-08-16 19:05:00總比要大家全部都直接上main好吧
作者:
newbout ( nb 但沒有 a )
2024-08-16 19:30:00看公司吧,我之前公司是小接案公司,功能在 dev branch上沒什麼問題就給客戶看了
作者:
wulouise (在線上!=在電腦前)
2024-08-16 20:10:00github pr不適合per commit review但是交流還是方便很多沒錯啦
作者:
Ekmund (是一隻小叔)
2024-08-16 20:20:00就風格不同吧 我遇過不同team 有的會發 有的直上的公司也遇過流程上會先後經過design review、code review這我就覺得發不發都還好當然完全不管的 應該連討論都不用啦
作者:
MoonCode (MoonCode)
2024-08-16 20:33:00不用 pr 就怕人工合併的時候被加料 去跟誰解釋 這資安扣分吧
作者: dog30111 (安) 2024-08-16 20:39:00
是我的話會站出來推動這件事,是個展現軟實力的機會
作者:
DrTech (竹科管理處網軍研發人員)
2024-08-16 20:50:00重點是程式碼品質有在管。而不是各種花俏,形式化的流程。程式碼品質,有在管PR可有可無。程式碼品質沒在管,再多review與流程,再多PR都沒用。
作者:
luke72 (ccc)
2024-08-16 21:45:00留comment給開發者,嗯,很多公司開發者就是你自己啊就算是上市大公司,常常功能切很細,repo還是只有你在做跨部門合作的repo發PR,但一人兩人的何必拘泥於這個
作者:
peter98 (新兵)
2024-08-16 21:50:00不會
作者:
luke72 (ccc)
2024-08-16 21:51:00我也見過一人repo走git flow,merge還要兩人簽核才能過然後某天半夜出bug要緊急修復,找不到人簽核….只好動用admin權限先砍了他的policy再說
作者:
peter98 (新兵)
2024-08-16 22:13:00樓上luke說的就是標準的系統爛、沒做好,Code review系統應該要有個override & merge
作者:
netburst (133 134 592)
2024-08-16 22:56:00沒用ftp就萬幸了
作者:
luke72 (ccc)
2024-08-17 00:01:00Code review & QA只是降低錯誤發生,不是免疫啦意外就是過去從未想過的狀況,能看出的就不是意外了一個team兩三個人,有幾十個小repo很常見吧我想講的就只是,大型repo的管理方法,並不是小型也適用
作者:
wulouise (在線上!=在電腦前)
2024-08-17 00:17:00github pr預設你一次全看,一條條看commit很麻煩..不過這就是設計理念不同的差異,至少還能多條看就很好了
脫褲子放屁而已 PR跟反對理由根本不衝突 單純不會用會覺得卡通常就只是把git當備份機制 習慣想怎麼改就怎麼改 垃圾進main後又跟部屬環境不一致 隨時想魔改rollback
一條一條看不是就按next commit就好了嗎=_=麻煩在哪
作者:
a731977 (卡哇邦卡)
2024-08-17 01:54:00不會
作者:
angusyu (〒△〒)
2024-08-17 02:30:00下篇文章:為什麼不用Github
作者:
acgotaku (otaku)
2024-08-17 05:01:00就沒 peer 可以 review 呀 我自己做自己的專案也懶得發
作者: poison5566 (已中毒) 2024-08-17 05:45:00
規模太小的團隊就容易沒有吧
作者: Firemaples (秋憶楓紅) 2024-08-17 08:26:00
遇過不 review,開發不切 branch,全靠人力 QA 管品質的公司有 review 已經很不錯了
作者: qazwsx12 (SexAnimal) 2024-08-17 10:24:00
我覺得質疑的也很怪..有這功能為啥不用,沒有缺點都是優點啊!
作者:
wulouise (在線上!=在電腦前)
2024-08-17 13:11:00他一條render一次沒辦法快速切吧,有辦法設定請告訴我..
作者:
Arbin (路人_Lv菜逼八)
2024-08-17 15:36:00還在用SVN的公司:
作者: yamagishi (山岸刑務官) 2024-08-17 15:49:00
原PO是在說優點那麼多又沒甚麼麻煩怎麼不用PR
有 review 就贏了,多的是開 PR 直接 approve
你可以直接跟對方討論優缺點 回來這裡優越發一篇是要幹嘛
作者:
pot1234 (鍋子)
2024-08-17 21:46:00gerrit好像沒用pr
作者:
LiebeLion (IchLiebeDich)
2024-08-17 21:56:00有commit就能review啊在branch code一樣可以留comment
作者:
qrtt1 (有些事,有時候。。。)
2024-08-17 22:55:00有 PR 才好接自動測試或是相關的 workflow
作者:
iamshiao (CircleHsiao)
2024-08-17 23:55:00既然都要 review 了,用 PR 比較方便吧
作者: luappc (luappc) 2024-08-18 18:04:00
待過使用TFS+Git的公司,走Git flow,每個同事分支權限開到最大,通常都自己直接Merge develop給QA測試,根本沒人用過TFS內建的PR功能,出問題再用git blame查是被誰改的
作者: smch (打哈哈) 2024-08-19 08:36:00
有review就不錯了
前公司沒在review 用SVN 沒在開分支全部人往主線傳QQ
作者: Hitmear (屍殌化液) 2024-08-19 13:21:00
聽起來是svn workflow,這就習慣而已又沒對錯
小公司還在給你玩官僚那套那早倒了沒事找事做是大公司賺錢後沒事幹的特權
作者: ashlikewing 2024-08-20 20:49:00
有了PR比較好review 是什麼概念
作者:
NDark (溺於黑暗)
2024-08-20 21:00:00部分的工程師偏好用文字溝通 也許一來一往比較有生產力
作者: mm58307533 2024-08-24 19:17:00
一堆根本不review的公司也發PR
作者:
knives 2024-08-25 16:56:00重要的是把事情做好
作者: MonyemLi (life) 2024-08-28 18:46:00
各種公司都有,看你可否接受