看板 Soft_Job作者 SaibuKani (西部のカニ)標題 [討論] Leader都把程式碼進到master時間 Sun Oct 8 03:00:33 2023
幫學弟問,原文照 po:
「我在台灣某網通大廠,第一份工作
leader 都把master 當自己的 branch
進 code 不用別人 review
常常把自己不能動的 code 推進去
或是偷偷改 lib 都不講
code進了不能跑,還被說效率不好
反應了也會被罵
平常會叫我去解釋 PR 寫了什麼
前陣子還要我教他 python decorator 是什麼
現在也會把 venv 跟 vm 的檔案推上去
每次 commit 都幾千個檔案
講了也被罵
有次我週五請病假
結果他晚上打電話來說要code review
跟我說以為是小病,沒差
他一直說業界都這樣做事
請問板上大大,真的是這樣嗎 」
--
Sent from nPTT on my
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 111.241.140.209 (臺灣)
※ 作者: SaibuKani 2023-10-08 03:00:33
※ 文章代碼(AID): #1b8QhJUU (Soft_Job)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1696705235.A.79E.html
推 Apache: 看起來你們沒有code review系統 也沒有CICD
雖然有導入可能也會沒開設定 還是可以自己進code2F 10/08 03:41
推 s860134: 切記 業界沒有這樣
正常公司不會這樣 這種只能靠你自立自強
外商也有這種 程度沒到大學畢業的6F 10/08 04:19
推 IhateOGC: 有些人單純嘴砲能力上位的
勸你還是快閃9F 10/08 04:51
→ Apache: 不懂技術不一定是問題 但不懂卻碰技術就是問題11F 10/08 04:54
噓 vongola12324: 奇文觀賞
每個業界都這樣幹那還得了
平衡推
順便偷問一下是哪一間12F 10/08 05:25
推 buyfood: 我碰過類似的,做三個月果斷烙跑17F 10/08 07:35
推 yamakazi: Jenkins gerrit 這種免費的CICD自己串一串不會?這種組內自己就可以搞定的,覺得看不慣就自己建流程是多難?18F 10/08 07:41
推 yamakazi: 全部都要等到公司準備好,那事情就不用做了21F 10/08 07:44
推 hsuchengmath: 建一個integration test 讓你主管不能推有問題的扣上去啊22F 10/08 07:49
→ peter98: 我認真回答你~你主管滿鳥的~他在科技業主管群裡的pr大概只有2524F 10/08 08:03
推 NDark: 拉出一個dev/release, 放棄已經存在的master讓他玩沙26F 10/08 08:32
→ luckid: 好奇是哪一間27F 10/08 08:46
推 qwe70302: 有些leader根本連code都不會寫,確實有pr2530F 10/08 09:36
→ peter98: 有 一定有更爛的主管幫原文說的這位稱到25 XD
我之前隔壁組~空降一個鳥主管到今年初2年半~底下的人換了一輪半~最後他的主管職位被他老闆拔掉~最後兩位從他手底轉組的人跟主管的主管說: 爛死了這甚麼主管?怎麼選的?
那你說原文這位老闆有沒有PR25?32F 10/08 10:02
推 lturtsamuel: 沒有code review就算了,跑不動的代碼竟然能進到master?世界奇觀37F 10/08 10:29
推 justaID: 我覺得塊陶 一個leader沒有開發制度也沒有善待下屬的氣度 在他底下只是浪費時間 不健康的成長 除非你能上達天聽 否則更上層的老闆只會信他不會信你 捅包你要背鍋 就算你做得很好 leader也只會收割你的功績
一個不適任的leader能上位可以有100種時光背景原因 一旦上位成功 再爛也沒那麼容易被拉下 尤其台廠很看重面子
所以我覺得只有塊陶一條路
*時空背景原因40F 10/08 10:44
推 luke72: 網通廠這樣的leader很正常,因為程度正常的早跑了48F 10/08 11:32
推 touurtn: 網通的純軟水準都不怎樣 都點在OS或計組50F 10/08 12:03
推 doranako: 比較常見網通跟系統廠是大家都直接進master,公平51F 10/08 12:23
推 luke72: 並沒有,我待過那麼多年沒見過leader懂os跟計組的52F 10/08 12:23
推 abccbaandy: 很正常吧,多的是把git當FTP用的,commit都無腦全選53F 10/08 12:25
→ luke72: Leader都是政治跟文學精通
懂技術的人都因為政治不行而被鬥去更高薪的產業了54F 10/08 12:25
推 magic83v: 還以為是我發的 趕快level up 趕快跳槽才是真的56F 10/08 12:30
→ ppc: 公司問題,建議換工作。58F 10/08 15:02
噓 iceman5566: 紅明顯 台灣某新創獨角獸的 IT head 超常假日推 code 到 master,然後也不講,超靠北,該 head 是一個老人,聽說是靠關係空降的,只能說這種文化並不是偶見59F 10/08 15:05
推 mike8469: 快逃R
大學畢業的新人都不會這樣幹.....62F 10/08 15:14
推 johnny94: 業界才沒有這樣,聽他在唬爛67F 10/08 15:34
推 tamama: 聽起來很像Mo?a68F 10/08 15:47
推 justaID: 樓上認真?我還以為404制度和文化跟傳統台廠比會比較好69F 10/08 16:04
推 zeus85072: 認真 我聽到蠻多鬼故事都是404出來的70F 10/08 16:12
推 GymRat: 鬼故事怎麼都很像71F 10/08 16:26
→ superpandal: 樓上說的是 混蛋其實蠻多的 這些人為了自己的生存可以無所不用其極 其實就是刻意亂凸顯你的錯誤和缺點這波操作下沒有問題都會變有問題 打訊息差逼你遷就他說回分支問題 多數人都是一人專案才直接用master 當72F 10/08 17:03
→ superpandal: 然不一定是這種狀況
我都是直接當ftp用 畢竟管理層的不雞掰的很少 被惡意針對還要替他們管理著想就太奴了 如果是好人當然就弄到好77F 10/08 17:17
→ DrTech: 很爛沒錯。但片面之言看看就好。如果沒上產品,你master怎麼改,其實都沒差。
真正有上產品,通常最後程式碼也未必在master分支。很多事情看表面沒意義。
最後程式碼的owner是誰?需要品質嗎? 如果owner不是你,也不需要品質,那你沒權限。
真的別把master分支當作多重要的一件事情,玩久了你就知道了,死守絕對的標準,還不如變通一下,另外拉其他分支自己玩就好。Release絕對沒有規定要在master才叫專業。81F 10/08 17:31
→ loadingN: 對啦,不是release的,沒什麼關係,但既然沒review就在local自己玩啊,推上去有意義嗎...90F 10/08 17:43
→ superpandal: 至於業界是不是都這樣 我只能說沒見過倉庫是很整齊乾凈的 或許誤commit 或許有疙瘩的commit
開源倉庫都不一定乾凈了 更別說職場混亂狀況92F 10/08 17:45
→ DrTech: 我待過標準規則一堆,做錯了要扣薪水,完整乾淨的倉庫。但我不認同管理repository有一個適用於所有公司部門的標準。 存在即是合理,程式碼或repository爛,還能繼續領薪水,代表該單位的獲利價值,或專業根本不在此而已。
專業當然很重要。但不專業的人竟然可以混得更好。通常是,你認為的專業,跟別人認為的專業不同而已。
別人又不靠乾淨的master分支賺錢時,這件事情就真不重要了。95F 10/08 17:51
→ superpandal: 應該沒公司靠乾凈軟體倉庫賺錢 會這樣要求的多半是管理職身在其位但想節省自己的心力搞的 但現實不可能103F 10/08 18:01
→ airtsubasa: 這不就跟變數命名 檔案命名一樣,誰是老大105F 10/08 18:03
→ superpandal: 節省心力有更好的方法就是 git不萬能 無法解決不會管理的人的管理問題106F 10/08 18:06
噓 typeking: 想也知道不可能業界都這樣,你還會專程上來發文問就知道你判斷能力也有待加強
繼續待在這公司就好108F 10/08 19:30
推 fu3mo6: 把default branch 改成 main113F 10/08 19:45
→ Apache: 確實 master不夠inclusive114F 10/08 19:59
推 feathergod: 會這樣做事 就是不靠程式碼賺錢 反正寫得爛 沒有規矩 公司照樣賺 薪水照樣領 幹嘛認真弄117F 10/08 20:47
→ gpctv: 這真的還好,我還聽過某金融連branch 切都沒切的,還說公司專案很小,這個不需要,但....是公司的核心系統,版控不曉得在做三小的119F 10/08 20:52
→ knives: 有這種主管,就算有review機制也是枉然好嗎122F 10/08 21:03
推 abc0922001: 有版控總比沒版控好出事至少能追到犯人XDD124F 10/08 21:47
推 N91: 逃吧 沒能力的主管126F 10/08 22:00
推 viper9709: 跑不動的代碼進master www129F 10/08 22:38
推 ZakuSIN: 往好處想 "至少有版控"
不爽不要做131F 10/09 00:26
→ peter98: 反推那些說master不重要的,如果master不重要,那就代表他沒再用,這樣應該沒人鳥master才對,如果有人鳥,代表master還是有在用,這樣就不應該上動都不能動的code去
master,另外說的release,額,你們又知道release不會從master sync過去了嗎? 一堆歪理。
每個team開發流程確實不一樣,如果master真的沒人鳥,那原po抱怨master的事情幹嘛? 阿不就是有人用嗎? 邏輯不好工作寫程式或當經理管理應該也都不怎麼滴
除非原po他自己不知道master其實沒人在鳥~這樣要嘴原po我覺得就說得過去。133F 10/09 00:34
→ superpandal: 有人說到重點 問題是主管雙標
master當然有人在用阿 主管卡著用阿 這本來就是雙標問題 repo管的好不好是另外一回事
不是分支管理不重要 是很多公司來講看起來不重要 而且推commit本身就有機會有事故或習慣不同 要完美的倉庫狀態是不可能的 工具可以輔助管理 但無法避免管理還可以扯到寫程式如何如何 你沒上帝視角看不了這麼多即便人在你面前都不一定可以看的出來
有寫輪眼的人可能可以看出來一點點 但沒這麼強大143F 10/09 10:31
推 jsa1090: 有CI/CD的話 可以設定基本lint failed卡住不能merge 或是github直接設定沒有review不能merge
如果主管只是懶惰的話 寫個script讓他只要一行指令就可以git pull rebase from master and checkout new branch153F 10/09 10:58
→ superpandal: 是 但不可避免中間會有人為因素 你整條流水線因素都cover到才稍微可以 但未知總是災難之源157F 10/09 11:03
噓 B0988698088: 講master不重要的是在共三小 即使可以靈活運用也不是像這樣被老害亂用好嗎 抬三小槓159F 10/09 11:49
→ DrTech: master本來就不重要。搞不好人家是用main啊,搞不好人家release不在 master啊。何必執著於名稱?有什麼國際標準,或專業論文說master比較好? 玩得通透的人,本來就不是看分支名稱決定品質與流程好壞。
而且如果master 沒有 release 作用, 為什麼master分支不能被當成程式碼暫存用途? 那個國際標準或專家說master分支不能用來暫存程式碼的? 歡迎提出證據與事實啊。
master本來就可以當成個人開發用的暫時分支。git本來就是這樣設計的。至於master分支,要不要當release分支,根本沒標準過。162F 10/09 15:16
→ Csongs: 原po意思是沒有release 分支吧
master 當release 分支,卻被leader當個人分支用173F 10/09 16:02
推 JoaoWang: 快逃 幾千個檔案有問題會看到死175F 10/09 16:25
--