[討論] 不發 PR 的公司會很怪嗎

作者: 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
有沒有一種可能是你見過的世面太少
作者: surfingbboy (科科)   2024-08-16 13:11:00
不會
作者: t19960804 (泥好嗎)   2024-08-16 13:15:00
作者: stepnight (桃卡武康)   2024-08-16 13:15:00
就是用不用這功能而已,做得也沒哪裡不一樣還是你覺得用什麼需求一定要用到PR才能做到
作者: Newtype (你快樂所以我快樂)   2024-08-16 13:15:00
至少有review了
作者: t19960804 (泥好嗎)   2024-08-16 13:18:00
是覺得有發pr比較正式吧
作者: qoo1991 (笨哲)   2024-08-16 13:20:00
Linus 也沒用PR 該怎麼辦
作者: mercurycgt68 (發芽的吉它手)   2024-08-16 13:26:00
trunk based development:
作者: bear1414 (story)   2024-08-16 13:27:00
方法(any)是靈活的 本質(review)才是重要的
作者: abc0922001 (中士abc)   2024-08-16 13:29:00
肯定有段故事的
作者: Imin0905 (Imin0905)   2024-08-16 13:32:00
有review就不錯了吧…
作者: feathergod (嗚嗚)   2024-08-16 13:35:00
至少有review 前公司不review還會在production branch開發
作者: MoonCode (MoonCode)   2024-08-16 13:37:00
沒有特別說明為何這樣做的話 就是雷
作者: NDark (溺於黑暗)   2024-08-16 13:38:00
PR只是一種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才是繞路。
作者: answermangtr (你今天抓了嘛)   2024-08-16 13:54:00
只是流程不一樣而已 還是有review
作者: shooter555 (shooter)   2024-08-16 14:02:00
沒有review就不用PR MR啦
作者: nh60211as   2024-08-16 14:03:00
但有發 PR 這流程會讓 review 變得輕鬆點 => 不一定
作者: NDark (溺於黑暗)   2024-08-16 14:09:00
同樓上
作者: abccbaandy (敏)   2024-08-16 14:36:00
有真review就屌打大部分公司了...
作者: ssccg (23)   2024-08-16 14:47:00
看起來只是你習慣用github的UI而已
作者: bheegrl   2024-08-16 15:04:00
大家有默契就好了
作者: chopinmozart (aha)   2024-08-16 15:40:00
Real man test on production
作者: luke72 (ccc)   2024-08-16 15:48:00
團隊才幾個人發PR是能多賺錢嗎?repo搞不好是單人開發一堆新手看了廣告文,就想拿5000人團隊制度套到5人團隊
作者: zxc8787 (摸斗哈壓庫)   2024-08-16 15:56:00
git的具體使用流程應該是配合公司吧有pr就有pr,沒有也不會怎樣吧
作者: abc0922001 (中士abc)   2024-08-16 16:16:00
有可能剛學會怎麼用git,沒時間也沒心力測試這個流程
作者: wei115 (ㄎㄎ)   2024-08-16 16:25:00
pr是github的功能吧?如果只是把github當git server沒pr也ok
作者: jackflu (jackflu)   2024-08-16 16:36:00
我覺得上面部份人其實不懂 PR,所以看不懂你的納悶,哈哈
作者: happy8649 (Hao)   2024-08-16 16:38:00
我看完留言想法也跟樓上一樣
作者: alan3100 (BOSS)   2024-08-16 17:43:00
pr都不懂別想說git flow自己管是多會管理拉..
作者: iamOsaka (歐沙卡)   2024-08-16 17:47:00
小團隊還好吧 如果一個repo有上百個人在開發哪可能非用不行
作者: moom50302 (武林三羚鱷)   2024-08-16 18:00:00
內文加留言 滿滿的工程師相輕
作者: henrylin8086 (小木)   2024-08-16 18:44: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上沒什麼問題就給客戶看了
作者: TSMCfabXX (台積新產品)   2024-08-16 19:38:00
一人開發 沒有大家
作者: wulouise (在線上!=在電腦前)   2024-08-16 20:10:00
github 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
是我的話會站出來推動這件事,是個展現軟實力的機會
作者: wd122344556 (我是LLK)   2024-08-16 20:46: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:00
Code review & QA只是降低錯誤發生,不是免疫啦意外就是過去從未想過的狀況,能看出的就不是意外了一個team兩三個人,有幾十個小repo很常見吧我想講的就只是,大型repo的管理方法,並不是小型也適用
作者: wulouise (在線上!=在電腦前)   2024-08-17 00:17:00
github pr預設你一次全看,一條條看commit很麻煩..不過這就是設計理念不同的差異,至少還能多條看就很好了
作者: alan3100 (BOSS)   2024-08-17 00:58:00
脫褲子放屁而已 PR跟反對理由根本不衝突 單純不會用會覺得卡通常就只是把git當備份機制 習慣想怎麼改就怎麼改 垃圾進main後又跟部屬環境不一致 隨時想魔改rollback
作者: happy8649 (Hao)   2024-08-17 01:17:00
一條一條看不是就按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一次沒辦法快速切吧,有辦法設定請告訴我..
作者: geoege022702 (raysquare)   2024-08-17 14:32:00
ㄤㄧ
作者: Arbin (路人_Lv菜逼八)   2024-08-17 15:36:00
還在用SVN的公司:
作者: yamagishi (山岸刑務官)   2024-08-17 15:49:00
原PO是在說優點那麼多又沒甚麼麻煩怎麼不用PR
作者: Phenomenon (Kuma)   2024-08-17 18:00:00
有 review 就贏了,多的是開 PR 直接 approve
作者: B0988698088 (廢文少女小円♥)   2024-08-17 18:36:00
你可以直接跟對方討論優缺點 回來這裡優越發一篇是要幹嘛
作者: qpowjohn (pose)   2024-08-17 19:07:00
我的感覺就是早期用SVN,後來轉移到git的公司
作者: pot1234 (鍋子)   2024-08-17 21:46:00
gerrit好像沒用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查是被誰改的
作者: notimenofree (逼逼逼逼逼)   2024-08-19 05:59:00
你可以問主管啊問我們怎麼知道
作者: smch (打哈哈)   2024-08-19 08:36:00
有review就不錯了
作者: bean90638 (bean)   2024-08-19 09:06:00
前公司沒在review 用SVN 沒在開分支全部人往主線傳QQ
作者: MonkeyCL (猴總召)   2024-08-19 11:01:00
真的 有review就不錯了
作者: lin70208 (漂泊大大)   2024-08-19 12:47:00
你問一下主管就知道了阿...
作者: Hitmear (屍殌化液)   2024-08-19 13:21:00
聽起來是svn workflow,這就習慣而已又沒對錯
作者: becca945 (頻果芽子)   2024-08-19 21:24:00
沒拿usb傳給你不錯了
作者: lovebridget (= =")   2024-08-19 22:37:00
小公司還在給你玩官僚那套那早倒了沒事找事做是大公司賺錢後沒事幹的特權
作者: ashlikewing   2024-08-20 20:49:00
有了PR比較好review 是什麼概念
作者: NDark (溺於黑暗)   2024-08-20 21:00:00
部分的工程師偏好用文字溝通 也許一來一往比較有生產力
作者: Hwangloveyu (PUCsie099)   2024-08-23 19:59: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
各種公司都有,看你可否接受

Links booklink

Contact Us: admin [ a t ] ucptt.com