※ 文章網址: https://www.ptt.cc/bbs/Tech_Job/M.1690424643.A.A9A.html
推 yytseng : 不是同team就可以亂改的，我們家分工就是給owner改 07/27 10:43
→ yytseng : 你發現有bug就mail給owner改就好，自己改會被公幹的 07/27 10:44
→ yytseng : 不是給你老闆那種公幹喔，是砲到處長全team公幹 07/27 10:44
→ yytseng : 你把社群那種 open source 精神跟公司團隊分工搞錯 07/27 10:45
→ yytseng : 有時候 bug在那邊是人家故意放著的，要跟前版相容 07/27 10:47
→ yytseng : 你一改影響到大票人的工作，畢竟你不是owner不清處 07/27 10:48
推 haoto : 別碰我的垃圾.jpg 07/27 10:48
推 wahaha279 : pr不適合，owner拒絕就好，幹嘛不爽 07/27 10:52
→ yamakazi : 改了push又不是馬上會被merge，PR不同意就好，還要 07/27 10:56
→ yamakazi : 全組公幹真可怕 07/27 10:56
推 FXW11314 : 開源社群也是先發PR等review，誰會直接push= = 07/27 10:57
→ yamakazi : 更何況還有QA測過才能merge，要是一個外人一改動馬 07/27 10:59
→ yamakazi : 上影響到一堆人那才有問題吧 07/27 10:59
→ FXW11314 : 幫忙debug很好但不講就硬上fix很雷 07/27 10:59
→ yamakazi : 不Push上去要怎麼發PR review? 不是一Push就會合進 07/27 11:00
→ yamakazi : 去耶 07/27 11:00
→ yamakazi : 有CICD的話，push之後起碼要有人approve+UT+自動化 07/27 11:01
→ yamakazi : 測項通過才有可能merge。有的還要加上QA測過 07/27 11:01
→ yamakazi : 才會merge。哪家公司一push就會merge啊？ 07/27 11:02
推 yytseng : 我們不是做軟體的，一改的確會很多tool去call用 07/27 11:03
→ yamakazi : Perforce有可能，一commit就會合進去超爛 07/27 11:04
→ yamakazi : Git你CICD有串好絕對不會一push就merge進master 07/27 11:04
→ yamakazi : 我之前在design house的時候，連編譯不過的扣都可以 07/27 11:06
→ yamakazi : 合進master，厲害了吧，perforce就是這麼爛 07/27 11:06
→ raxt577 : 同team可以亂改？出事會是找改的人還是找owner?想清 07/27 11:17
→ raxt577 : 楚好嗎？ 07/27 11:17
推 magamanzero : 會寫在KPI上就不是熱心了 這根本沒討論價值 07/27 11:19
→ yamakazi : 我發現這是軟體方法論的差異，有的公司是一個人負責 07/27 11:27
→ yamakazi : 一部分，有的公司是一個team負責一大部分，任何人在 07/27 11:27
→ yamakazi : 這個team都可以改這部分的代碼。後者叫集體代碼共有 07/27 11:27
→ yamakazi : 制 07/27 11:27
→ yamakazi : https://i.imgur.com/kXtexge.jpg 07/27 11:29
→ yamakazi : 找owner基本上就不是走集體所有制，我待過的公司都 07/27 11:30
→ yamakazi : 是走集體所有制，所以沒有這種問題 07/27 11:30
推 mooto : 那不是performance 的問題，是把關的問題，你讓user 07/27 11:31
→ mooto : 沒qc就可以commit 07/27 11:31
→ yamakazi : 走集體制的公司，基本上就不會有什麼owner的問題， 07/27 11:40
→ yamakazi : 所以底下意見才這麼兩歧，因為大家價值觀不同 07/27 11:40
推 hcwang1126 : 其實不管是不是owner merge前都是要被review 這可能 07/27 11:58
→ hcwang1126 : 才是觀念上的最大分歧 07/27 11:58
→ hcwang1126 : 基於此 小弟的公司都不是以上的制度 07/27 11:59
→ hcwang1126 : Review不能的人或code 很常被噴爛 只能說誰叫他是開 07/27 12:00
→ hcwang1126 : 國元老 07/27 12:00
→ flash5408 : 大開眼界 我以為軟體公司絕大部分的程式碼都是全公 07/27 12:04
→ flash5408 : 司看得到的 想改哪就送cr給owner. 設一堆奇怪權限 07/27 12:04
→ flash5408 : 幹嘛 07/27 12:04
→ yamakazi : 我也以為是這樣啊，才會有疑問是吵什麼owner? 07/27 12:06
推 wulouise : onwer review過不就好？要owner改應該是bug很少 07/27 12:13
推 NCTUFAIWEN : 笑死 我們master code編譯不過根本是日常 編譯測試 07/27 12:14
→ NCTUFAIWEN : 是在merge進去才做的XDD 07/27 12:14
→ flash5408 : 是矽谷特別還是台灣有自己的玩法 我怎麼常在這裡看 07/27 12:16
→ flash5408 : 到一堆無法理解的事情 07/27 12:16
→ dildoe : 一流大企業都是垃圾桶不開蓋 再找一群人來喊臭 保 07/27 12:18
→ NCTUFAIWEN : CI/CD根本就軟體公司才會重視 產品是硬體的誰在意阿 07/27 12:18
→ dildoe : 證老闆不敢動 西西XD 07/27 12:19
→ NCTUFAIWEN : 連code review都不一定有 07/27 12:19
推 yudofu : 原文寫revise，光這樣也看不出來他是自己動手改的還 07/27 13:14
→ yudofu : 是透過公司流程，一堆人在那邊腦補，沒有寫的就不要 07/27 13:15
→ yudofu : 自己亂加油添醋，看東西說不也是工程師基礎嗎，笑死 07/27 13:16
推 Wishmaster : 我也是覺得,給他們merge權就好,不要自己merge進去 07/27 14:18
→ Wishmaster : 應該還好吧,我發了pr,merge不merge是你的事 07/27 14:19
推 Litfal : 有集體共有制，也有僅知原則阿，要看公司 07/28 08:33
推 wcre : 前一篇回文正解，但個人喜歡你的建議 07/28 09:22