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

yytseng 07/27 10:43不是同team就可以亂改的，我們家分工就是給owner改

yytseng 07/27 10:44你發現有bug就mail給owner改就好，自己改會被公幹的

yytseng 07/27 10:44不是給你老闆那種公幹喔，是砲到處長全team公幹

yytseng 07/27 10:45你把社群那種 open source 精神跟公司團隊分工搞錯

yytseng 07/27 10:47有時候 bug在那邊是人家故意放著的，要跟前版相容

yytseng 07/27 10:48你一改影響到大票人的工作，畢竟你不是owner不清處

haoto 07/27 10:48別碰我的垃圾.jpg

wahaha279 07/27 10:52pr不適合，owner拒絕就好，幹嘛不爽

yamakazi 07/27 10:56改了push又不是馬上會被merge，PR不同意就好，還要

yamakazi 07/27 10:56全組公幹真可怕

FXW11314 07/27 10:57開源社群也是先發PR等review，誰會直接push= =

yamakazi 07/27 10:59更何況還有QA測過才能merge，要是一個外人一改動馬

yamakazi 07/27 10:59上影響到一堆人那才有問題吧

FXW11314 07/27 10:59幫忙debug很好但不講就硬上fix很雷

yamakazi 07/27 11:00不Push上去要怎麼發PR review? 不是一Push就會合進

yamakazi 07/27 11:00去耶

yamakazi 07/27 11:01有CICD的話，push之後起碼要有人approve+UT+自動化

yamakazi 07/27 11:01測項通過才有可能merge。有的還要加上QA測過

yamakazi 07/27 11:02才會merge。哪家公司一push就會merge啊？

yytseng 07/27 11:03我們不是做軟體的，一改的確會很多tool去call用

yamakazi 07/27 11:04Perforce有可能，一commit就會合進去超爛

yamakazi 07/27 11:04Git你CICD有串好絕對不會一push就merge進master

yamakazi 07/27 11:06我之前在design house的時候，連編譯不過的扣都可以

yamakazi 07/27 11:06合進master，厲害了吧，perforce就是這麼爛

raxt577 07/27 11:17同team可以亂改？出事會是找改的人還是找owner?想清

raxt577 07/27 11:17楚好嗎？

magamanzero 07/27 11:19會寫在KPI上就不是熱心了 這根本沒討論價值

yamakazi 07/27 11:27我發現這是軟體方法論的差異，有的公司是一個人負責

yamakazi 07/27 11:27一部分，有的公司是一個team負責一大部分，任何人在

yamakazi 07/27 11:27這個team都可以改這部分的代碼。後者叫集體代碼共有

yamakazi 07/27 11:27制

yamakazi 07/27 11:29https://i.imgur.com/kXtexge.jpg

yamakazi 07/27 11:30找owner基本上就不是走集體所有制，我待過的公司都

yamakazi 07/27 11:30是走集體所有制，所以沒有這種問題

mooto 07/27 11:31那不是performance 的問題，是把關的問題，你讓user

mooto 07/27 11:31沒qc就可以commit

yamakazi 07/27 11:40走集體制的公司，基本上就不會有什麼owner的問題，

yamakazi 07/27 11:40所以底下意見才這麼兩歧，因為大家價值觀不同

hcwang1126 07/27 11:58其實不管是不是owner merge前都是要被review 這可能

hcwang1126 07/27 11:58才是觀念上的最大分歧

hcwang1126 07/27 11:59基於此 小弟的公司都不是以上的制度

hcwang1126 07/27 12:00Review不能的人或code 很常被噴爛 只能說誰叫他是開

hcwang1126 07/27 12:00國元老

flash5408 07/27 12:04大開眼界 我以為軟體公司絕大部分的程式碼都是全公

flash5408 07/27 12:04司看得到的 想改哪就送cr給owner. 設一堆奇怪權限

flash5408 07/27 12:04幹嘛

yamakazi 07/27 12:06我也以為是這樣啊，才會有疑問是吵什麼owner?

wulouise 07/27 12:13onwer review過不就好？要owner改應該是bug很少

NCTUFAIWEN 07/27 12:14笑死 我們master code編譯不過根本是日常 編譯測試

NCTUFAIWEN 07/27 12:14是在merge進去才做的XDD

flash5408 07/27 12:16是矽谷特別還是台灣有自己的玩法 我怎麼常在這裡看

flash5408 07/27 12:16到一堆無法理解的事情

dildoe 07/27 12:18一流大企業都是垃圾桶不開蓋 再找一群人來喊臭 保

NCTUFAIWEN 07/27 12:18CI/CD根本就軟體公司才會重視 產品是硬體的誰在意阿

dildoe 07/27 12:19證老闆不敢動 西西XD

NCTUFAIWEN 07/27 12:19連code review都不一定有

yudofu 07/27 13:14原文寫revise，光這樣也看不出來他是自己動手改的還

yudofu 07/27 13:15是透過公司流程，一堆人在那邊腦補，沒有寫的就不要

yudofu 07/27 13:16自己亂加油添醋，看東西說不也是工程師基礎嗎，笑死

Wishmaster 07/27 14:18我也是覺得,給他們merge權就好,不要自己merge進去

Wishmaster 07/27 14:19應該還好吧,我發了pr,merge不merge是你的事

Litfal 07/28 08:33有集體共有制，也有僅知原則阿，要看公司

wcre 07/28 09:22前一篇回文正解，但個人喜歡你的建議