Re: [討論] 熱心或撈過界

看板Tech_Job作者 (大安吳彥祖)時間9月前 (2023/07/27 10:24), 編輯推噓14(14推 0噓 49→)
留言63則, 17人參與, 9月前最新討論串3/4 (看更多)
不想被別team review就鎖權限啊 權限ㄧ鎖不要說git pull Jira Wiki Bitbucket什麼的通通看不到 連docker環境都可以鎖權限不讓你pull 更別說那種需要機器才能跑的軟體 今天你不鎖權限，那不就代表同team? 那不就跟Linux kernel一樣open source。 幾乎任何人都可以報bug，push code。 你有看過網路開源軟體社群有人抱怨撈過界嗎？ 如果你是那兩人的二級主管那不就表示同team? 同team還有分那麼多你我哦 真的要分你我看你就再分組下去然後各組互鎖權限不給看 我看你這樣管理起來應該更爆幹麻煩吧？ ----- Sent from JPTT on my iPhone -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 1.200.69.152 (臺灣) ※ 文章網址: https://www.ptt.cc/bbs/Tech_Job/M.1690424643.A.A9A.html
推
07/27 10:43, 9月前 , 1F
不是同team就可以亂改的，我們家分工就是給owner改
07/27 10:43, 1F
→
07/27 10:44, 9月前 , 2F
你發現有bug就mail給owner改就好，自己改會被公幹的
07/27 10:44, 2F
→
07/27 10:44, 9月前 , 3F
不是給你老闆那種公幹喔，是砲到處長全team公幹
07/27 10:44, 3F
→
07/27 10:45, 9月前 , 4F
你把社群那種 open source 精神跟公司團隊分工搞錯
07/27 10:45, 4F
→
07/27 10:47, 9月前 , 5F
有時候 bug在那邊是人家故意放著的，要跟前版相容
07/27 10:47, 5F
→
07/27 10:48, 9月前 , 6F
你一改影響到大票人的工作，畢竟你不是owner不清處
07/27 10:48, 6F
推
07/27 10:48, 9月前 , 7F
別碰我的垃圾.jpg
07/27 10:48, 7F
推
07/27 10:52, 9月前 , 8F
pr不適合，owner拒絕就好，幹嘛不爽
07/27 10:52, 8F
→
07/27 10:56, 9月前 , 9F
改了push又不是馬上會被merge，PR不同意就好，還要
07/27 10:56, 9F
→
07/27 10:56, 9月前 , 10F
全組公幹真可怕
07/27 10:56, 10F
推
07/27 10:57, 9月前 , 11F
開源社群也是先發PR等review，誰會直接push= =
07/27 10:57, 11F
→
07/27 10:59, 9月前 , 12F
更何況還有QA測過才能merge，要是一個外人一改動馬
07/27 10:59, 12F
→
07/27 10:59, 9月前 , 13F
上影響到一堆人那才有問題吧
07/27 10:59, 13F
→
07/27 10:59, 9月前 , 14F
幫忙debug很好但不講就硬上fix很雷
07/27 10:59, 14F
→
07/27 11:00, 9月前 , 15F
不Push上去要怎麼發PR review? 不是一Push就會合進
07/27 11:00, 15F
→
07/27 11:00, 9月前 , 16F
去耶
07/27 11:00, 16F
→
07/27 11:01, 9月前 , 17F
有CICD的話，push之後起碼要有人approve+UT+自動化
07/27 11:01, 17F
→
07/27 11:01, 9月前 , 18F
測項通過才有可能merge。有的還要加上QA測過
07/27 11:01, 18F
→
07/27 11:02, 9月前 , 19F
才會merge。哪家公司一push就會merge啊？
07/27 11:02, 19F
推
07/27 11:03, 9月前 , 20F
我們不是做軟體的，一改的確會很多tool去call用
07/27 11:03, 20F
→
07/27 11:04, 9月前 , 21F
Perforce有可能，一commit就會合進去超爛
07/27 11:04, 21F
→
07/27 11:04, 9月前 , 22F
Git你CICD有串好絕對不會一push就merge進master
07/27 11:04, 22F
→
07/27 11:06, 9月前 , 23F
我之前在design house的時候，連編譯不過的扣都可以
07/27 11:06, 23F
→
07/27 11:06, 9月前 , 24F
合進master，厲害了吧，perforce就是這麼爛
07/27 11:06, 24F
→
07/27 11:17, 9月前 , 25F
同team可以亂改？出事會是找改的人還是找owner?想清
07/27 11:17, 25F
→
07/27 11:17, 9月前 , 26F
楚好嗎？
07/27 11:17, 26F
推
07/27 11:19, 9月前 , 27F
會寫在KPI上就不是熱心了 這根本沒討論價值
07/27 11:19, 27F
→
07/27 11:27, 9月前 , 28F
我發現這是軟體方法論的差異，有的公司是一個人負責
07/27 11:27, 28F
→
07/27 11:27, 9月前 , 29F
一部分，有的公司是一個team負責一大部分，任何人在
07/27 11:27, 29F
→
07/27 11:27, 9月前 , 30F
這個team都可以改這部分的代碼。後者叫集體代碼共有
07/27 11:27, 30F
→
07/27 11:27, 9月前 , 31F
制
07/27 11:27, 31F
→
07/27 11:29, 9月前 , 32F
07/27 11:29, 32F
→
07/27 11:30, 9月前 , 33F
找owner基本上就不是走集體所有制，我待過的公司都
07/27 11:30, 33F
→
07/27 11:30, 9月前 , 34F
是走集體所有制，所以沒有這種問題
07/27 11:30, 34F
推
07/27 11:31, 9月前 , 35F
那不是performance 的問題，是把關的問題，你讓user
07/27 11:31, 35F
→
07/27 11:31, 9月前 , 36F
沒qc就可以commit
07/27 11:31, 36F
→
07/27 11:40, 9月前 , 37F
走集體制的公司，基本上就不會有什麼owner的問題，
07/27 11:40, 37F
→
07/27 11:40, 9月前 , 38F
所以底下意見才這麼兩歧，因為大家價值觀不同
07/27 11:40, 38F
推
07/27 11:58, 9月前 , 39F
其實不管是不是owner merge前都是要被review 這可能
07/27 11:58, 39F
→
07/27 11:58, 9月前 , 40F
才是觀念上的最大分歧
07/27 11:58, 40F
→
07/27 11:59, 9月前 , 41F
基於此 小弟的公司都不是以上的制度
07/27 11:59, 41F
→
07/27 12:00, 9月前 , 42F
Review不能的人或code 很常被噴爛 只能說誰叫他是開
07/27 12:00, 42F
→
07/27 12:00, 9月前 , 43F
國元老
07/27 12:00, 43F
→
07/27 12:04, 9月前 , 44F
大開眼界 我以為軟體公司絕大部分的程式碼都是全公
07/27 12:04, 44F
→
07/27 12:04, 9月前 , 45F
司看得到的 想改哪就送cr給owner. 設一堆奇怪權限
07/27 12:04, 45F
→
07/27 12:04, 9月前 , 46F
幹嘛
07/27 12:04, 46F
→
07/27 12:06, 9月前 , 47F
我也以為是這樣啊，才會有疑問是吵什麼owner?
07/27 12:06, 47F
推
07/27 12:13, 9月前 , 48F
onwer review過不就好？要owner改應該是bug很少
07/27 12:13, 48F
推
07/27 12:14, 9月前 , 49F
笑死 我們master code編譯不過根本是日常 編譯測試
07/27 12:14, 49F
→
07/27 12:14, 9月前 , 50F
是在merge進去才做的XDD
07/27 12:14, 50F
→
07/27 12:16, 9月前 , 51F
是矽谷特別還是台灣有自己的玩法 我怎麼常在這裡看
07/27 12:16, 51F
→
07/27 12:16, 9月前 , 52F
到一堆無法理解的事情
07/27 12:16, 52F
→
07/27 12:18, 9月前 , 53F
一流大企業都是垃圾桶不開蓋 再找一群人來喊臭 保
07/27 12:18, 53F
→
07/27 12:18, 9月前 , 54F
CI/CD根本就軟體公司才會重視 產品是硬體的誰在意阿
07/27 12:18, 54F
→
07/27 12:19, 9月前 , 55F
證老闆不敢動 西西XD
07/27 12:19, 55F
→
07/27 12:19, 9月前 , 56F
連code review都不一定有
07/27 12:19, 56F
推
07/27 13:14, 9月前 , 57F
原文寫revise，光這樣也看不出來他是自己動手改的還
07/27 13:14, 57F
→
07/27 13:15, 9月前 , 58F
是透過公司流程，一堆人在那邊腦補，沒有寫的就不要
07/27 13:15, 58F
→
07/27 13:16, 9月前 , 59F
自己亂加油添醋，看東西說不也是工程師基礎嗎，笑死
07/27 13:16, 59F
推
07/27 14:18, 9月前 , 60F
我也是覺得,給他們merge權就好,不要自己merge進去
07/27 14:18, 60F
→
07/27 14:19, 9月前 , 61F
應該還好吧,我發了pr,merge不merge是你的事
07/27 14:19, 61F
推
07/28 08:33, 9月前 , 62F
有集體共有制，也有僅知原則阿，要看公司
07/28 08:33, 62F
推
07/28 09:22, 9月前 , 63F
前一篇回文正解，但個人喜歡你的建議
07/28 09:22, 63F
‣ 更多 yamakazi 的文章
文章代碼(AID): #1amTL3gQ (Tech_Job)
‣ 更多 yamakazi 的文章
文章代碼(AID): #1amTL3gQ (Tech_Job)