[討論] Leader都把程式碼進到master

作者: SaibuKani (SaibuKani)   2023-10-08 03:00:33
幫學弟問,原文照 po:

「我在台灣某網通大廠,第一份工作

leader 都把master 當自己的 branch
進 code 不用別人 review
常常把自己不能動的 code 推進去
或是偷偷改 lib 都不講

code進了不能跑,還被說效率不好
反應了也會被罵

平常會叫我去解釋 PR 寫了什麼
前陣子還要我教他 python decorator 是什麼

現在也會把 venv 跟 vm 的檔案推上去
每次 commit 都幾千個檔案
講了也被罵

有次我週五請病假
結果他晚上打電話來說要code review
跟我說以為是小病,沒差

他一直說業界都這樣做事
請問板上大大,真的是這樣嗎 」
作者: andy00andy (髮)   2023-10-08 03:11:00
這種能做到leader,還不逃?
作者: Apache (阿帕契)   2023-10-08 03:41:00
看起來你們沒有code review系統 也沒有CICD雖然有導入可能也會沒開設定 還是可以自己進code
作者: jonathan793 (pusheen cat)   2023-10-08 03:57:00
塊陶
作者: Firstshadow (IamCatづミ'_'ミづ)   2023-10-08 04:08:00
這該不會也是你leader的第一份吧
作者: s860134 (s860134)   2023-10-08 04:19:00
切記 業界沒有這樣正常公司不會這樣 這種只能靠你自立自強外商也有這種 程度沒到大學畢業的
作者: IhateOGC (我討厭)   2023-10-08 04:51:00
有些人單純嘴砲能力上位的勸你還是快閃
作者: Apache (阿帕契)   2023-10-08 04:54:00
不懂技術不一定是問題 但不懂卻碰技術就是問題
作者: vongola12324   2023-10-08 05:25:00
奇文觀賞每個業界都這樣幹那還得了平衡推順便偷問一下是哪一間
作者: zebra10029   2023-10-08 06:32:00
快逃
作者: buyfood (buyfood)   2023-10-08 07:35:00
我碰過類似的,做三個月果斷烙跑
作者: yamakazi (大安吳彥祖)   2023-10-08 07:41:00
Jenkins gerrit 這種免費的CICD自己串一串不會?這種組內自己就可以搞定的,覺得看不慣就自己建流程是多難?
作者: airtsubasa (偽學姊)   2023-10-08 07:44:00
製造業正常啦
作者: yamakazi (大安吳彥祖)   2023-10-08 07:44:00
全部都要等到公司準備好,那事情就不用做了
作者: hsuchengmath (AlbertHSU)   2023-10-08 07:49:00
建一個integration test 讓你主管不能推有問題的扣上去啊
作者: peter98 (新兵)   2023-10-08 08:03:00
我認真回答你~你主管滿鳥的~他在科技業主管群裡的pr大概只有25
作者: NDark (溺於黑暗)   2023-10-08 08:32:00
拉出一個dev/release, 放棄已經存在的master讓他玩沙
作者: luckid (luckid)   2023-10-08 08:46:00
好奇是哪一間
作者: loadingN (sarsaparilla)   2023-10-08 08:58:00
不是業界 是三流系統廠
作者: sniper2824 (月夜)   2023-10-08 09:01:00
這樣pr還有25喔
作者: qwe70302 (為何一到90分就會輸)   2023-10-08 09:36:00
有些leader根本連code都不會寫,確實有pr25
作者: wulouise (在線上!=在電腦前)   2023-10-08 09:53:00
嗯,一個字跑
作者: peter98 (新兵)   2023-10-08 10:02:00
有 一定有更爛的主管幫原文說的這位稱到25 XD我之前隔壁組~空降一個鳥主管到今年初2年半~底下的人換了一輪半~最後他的主管職位被他老闆拔掉~最後兩位從他手底轉組的人跟主管的主管說: 爛死了這甚麼主管?怎麼選的?那你說原文這位老闆有沒有PR25?
作者: lturtsamuel (港都都教授)   2023-10-08 10:29:00
沒有code review就算了,跑不動的代碼竟然能進到master?世界奇觀
作者: sniper2824 (月夜)   2023-10-08 10:38:00
真可怕= =
作者: justaID (快樂崇拜)   2023-10-08 10:44:00
我覺得塊陶 一個leader沒有開發制度也沒有善待下屬的氣度 在他底下只是浪費時間 不健康的成長 除非你能上達天聽 否則更上層的老闆只會信他不會信你 捅包你要背鍋 就算你做得很好 leader也只會收割你的功績一個不適任的leader能上位可以有100種時光背景原因 一旦上位成功 再爛也沒那麼容易被拉下 尤其台廠很看重面子所以我覺得只有塊陶一條路*時空背景原因
作者: luke72 (ccc)   2023-10-08 11:32:00
網通廠這樣的leader很正常,因為程度正常的早跑了
作者: johnbill (cj鐘鐘)   2023-10-08 11:52:00
快逃
作者: touurtn (vv)   2023-10-08 12:03:00
網通的純軟水準都不怎樣 都點在OS或計組
作者: doranako (真愛無限)   2023-10-08 12:23:00
比較常見網通跟系統廠是大家都直接進master,公平
作者: luke72 (ccc)   2023-10-08 12:23:00
並沒有,我待過那麼多年沒見過leader懂os跟計組的
作者: abccbaandy (敏)   2023-10-08 12:25:00
很正常吧,多的是把git當FTP用的,commit都無腦全選
作者: luke72 (ccc)   2023-10-08 12:25:00
Leader都是政治跟文學精通懂技術的人都因為政治不行而被鬥去更高薪的產業了
作者: magic83v (R7)   2023-10-08 12:30:00
還以為是我發的 趕快level up 趕快跳槽才是真的
作者: WaterLengend (Leeeeeeeeooooooo)   2023-10-08 12:46:00
遇過類似的,結果看那主管的領英越混越差,可悲
作者: ppc ( )   2023-10-08 15:02:00
公司問題,建議換工作。
作者: iceman5566 (iceman5566)   2023-10-08 15:05:00
紅明顯 台灣某新創獨角獸的 IT head 超常假日推 code 到 master,然後也不講,超靠北,該 head 是一個老人,聽說是靠關係空降的,只能說這種文化並不是偶見
作者: mike8469 (mike8469)   2023-10-08 15:14:00
快逃R大學畢業的新人都不會這樣幹.....
作者: nayeonmywife (sanamywife)   2023-10-08 15:18:00
Wow
作者: xsdferty035 (Free逛逛)   2023-10-08 15:21:00
不能跑還進==
作者: realbout (薩摩訶)   2023-10-08 15:22:00
這leader是不是台大
作者: johnny94 (32767)   2023-10-08 15:34:00
業界才沒有這樣,聽他在唬爛
作者: tamama (tamama)   2023-10-08 15:47:00
聽起來很像Mo?a
作者: justaID (快樂崇拜)   2023-10-08 16:04:00
樓上認真?我還以為404制度和文化跟傳統台廠比會比較好
作者: zeus85072 (deus)   2023-10-08 16:12:00
認真 我聽到蠻多鬼故事都是404出來的
作者: GymRat (鼠鼠有練過)   2023-10-08 16:26:00
鬼故事怎麼都很像
作者: superpandal   2023-10-08 17:03:00
樓上說的是 混蛋其實蠻多的 這些人為了自己的生存可以無所不用其極 其實就是刻意亂凸顯你的錯誤和缺點這波操作下沒有問題都會變有問題 打訊息差逼你遷就他說回分支問題 多數人都是一人專案才直接用master 當
作者: v86861062 (數字人:3)   2023-10-08 17:17:00
真慘
作者: superpandal   2023-10-08 17:17:00
然不一定是這種狀況我都是直接當ftp用 畢竟管理層的不雞掰的很少 被惡意針對還要替他們管理著想就太奴了 如果是好人當然就弄到好
作者: DrTech (竹科管理處網軍研發人員)   2023-10-08 17:31:00
很爛沒錯。但片面之言看看就好。如果沒上產品,你master怎麼改,其實都沒差。真正有上產品,通常最後程式碼也未必在master分支。很多事情看表面沒意義。最後程式碼的owner是誰?需要品質嗎? 如果owner不是你,也不需要品質,那你沒權限。真的別把master分支當作多重要的一件事情,玩久了你就知道了,死守絕對的標準,還不如變通一下,另外拉其他分支自己玩就好。Release絕對沒有規定要在master才叫專業。
作者: loadingN (sarsaparilla)   2023-10-08 17:43:00
對啦,不是release的,沒什麼關係,但既然沒review就在local自己玩啊,推上去有意義嗎...
作者: superpandal   2023-10-08 17:45:00
至於業界是不是都這樣 我只能說沒見過倉庫是很整齊乾凈的 或許誤commit 或許有疙瘩的commit開源倉庫都不一定乾凈了 更別說職場混亂狀況
作者: DrTech (竹科管理處網軍研發人員)   2023-10-08 17:51:00
我待過標準規則一堆,做錯了要扣薪水,完整乾淨的倉庫。但我不認同管理repository有一個適用於所有公司部門的標準。存在即是合理,程式碼或repository爛,還能繼續領薪水,代表該單位的獲利價值,或專業根本不在此而已。專業當然很重要。但不專業的人竟然可以混得更好。通常是,你認為的專業,跟別人認為的專業不同而已。別人又不靠乾淨的master分支賺錢時,這件事情就真不重要了。
作者: superpandal   2023-10-08 18:01:00
應該沒公司靠乾凈軟體倉庫賺錢 會這樣要求的多半是管理職身在其位但想節省自己的心力搞的 但現實不可能
作者: airtsubasa (偽學姊)   2023-10-08 18:03:00
這不就跟變數命名 檔案命名一樣,誰是老大
作者: superpandal   2023-10-08 18:06:00
節省心力有更好的方法就是 git不萬能 無法解決不會管理的人的管理問題
作者: typeking ( )   2023-10-08 19:30:00
想也知道不可能業界都這樣,你還會專程上來發文問就知道你判斷能力也有待加強繼續待在這公司就好
作者: yamagishi (山岸刑務官)   2023-10-08 19:32:00
問題是主管雙標吧
作者: Csongs (西歌)   2023-10-08 19:42:00
嗯嗯
作者: fu3mo6 (ㄚ龐)   2023-10-08 19:45:00
把default branch 改成 main
作者: Apache (阿帕契)   2023-10-08 19:59:00
確實 master不夠inclusive
作者: f26724309 (番薯)   2023-10-08 20:06:00
這也太鬼XD 哪家阿讓大家笑笑
作者: kimisme9386   2023-10-08 20:14:00
快逃啊
作者: feathergod (嗚嗚)   2023-10-08 20:47:00
會這樣做事 就是不靠程式碼賺錢 反正寫得爛 沒有規矩公司照樣賺 薪水照樣領 幹嘛認真弄
作者: gpctv (gpctv)   2023-10-08 20:52:00
這真的還好,我還聽過某金融連branch 切都沒切的,還說公司專案很小,這個不需要,但....是公司的核心系統,版控不曉得在做三小的
作者: knives   2023-10-08 21:03:00
有這種主管,就算有review機制也是枉然好嗎
作者: blazers08 (我是傳奇)   2023-10-08 21:37:00
快逃
作者: abc0922001 (中士abc)   2023-10-08 21:47:00
有版控總比沒版控好出事至少能追到犯人XDD
作者: bearkevin (bear)   2023-10-08 21:56:00
不逃?
作者: N91 (N91)   2023-10-08 22:00:00
逃吧 沒能力的主管
作者: jay123peter (蕭瑟風雅)   2023-10-08 22:07:00
這種沒同理心主官都很雷...
作者: drakd4d (NULL)   2023-10-08 22:24:00
夭壽
作者: viper9709 (阿達)   2023-10-08 22:38:00
跑不動的代碼進master www
作者: Mike1109 (黃金右手)   2023-10-08 22:45:00
心臟真大顆
作者: ZakuSIN (SIN)   2023-10-09 00:26:00
往好處想 "至少有版控"不爽不要做
作者: peter98 (新兵)   2023-10-09 00:34:00
反推那些說master不重要的,如果master不重要,那就代表他沒再用,這樣應該沒人鳥master才對,如果有人鳥,代表master還是有在用,這樣就不應該上動都不能動的code去master,另外說的release,額,你們又知道release不會從master sync過去了嗎? 一堆歪理。每個team開發流程確實不一樣,如果master真的沒人鳥,那原po抱怨master的事情幹嘛? 阿不就是有人用嗎? 邏輯不好工作寫程式或當經理管理應該也都不怎麼滴除非原po他自己不知道master其實沒人在鳥~這樣要嘴原po我覺得就說得過去。
作者: superpandal   2023-10-09 10:31:00
有人說到重點 問題是主管雙標master當然有人在用阿 主管卡著用阿 這本來就是雙標問題 repo管的好不好是另外一回事不是分支管理不重要 是很多公司來講看起來不重要 而且推commit本身就有機會有事故或習慣不同 要完美的倉庫狀態是不可能的 工具可以輔助管理 但無法避免管理還可以扯到寫程式如何如何 你沒上帝視角看不了這麼多即便人在你面前都不一定可以看的出來有寫輪眼的人可能可以看出來一點點 但沒這麼強大
作者: hakosaki (hakosaki)   2023-10-09 10:57:00
快跑....
作者: jsa1090 (xoxoxo24)   2023-10-09 10:58:00
有CI/CD的話 可以設定基本lint failed卡住不能merge 或是github直接設定沒有review不能merge如果主管只是懶惰的話 寫個script讓他只要一行指令就可以git pull rebase from master and checkout new branch
作者: superpandal   2023-10-09 11:03:00
是 但不可避免中間會有人為因素 你整條流水線因素都cover到才稍微可以 但未知總是災難之源
作者: B0988698088 (廢文少女小円♥)   2023-10-09 11:49:00
講master不重要的是在共三小 即使可以靈活運用也不是像這樣被老害亂用好嗎 抬三小槓
作者: bachelorwhc (單身老王)   2023-10-09 13:41:00
這板槓精真多
作者: DrTech (竹科管理處網軍研發人員)   2023-10-09 15:16:00
master本來就不重要。搞不好人家是用main啊,搞不好人家release不在 master啊。何必執著於名稱?有什麼國際標準,或專業論文說master比較好? 玩得通透的人,本來就不是看分支名稱決定品質與流程好壞。而且如果master 沒有 release 作用, 為什麼master分支不能被當成程式碼暫存用途? 那個國際標準或專家說master分支不能用來暫存程式碼的? 歡迎提出證據與事實啊。master本來就可以當成個人開發用的暫時分支。git本來就是這樣設計的。至於master分支,要不要當release分支,根本沒標準過。
作者: leonEEman (EE)   2023-10-09 15:44:00
笑死 這leader是剛畢業嗎
作者: Csongs (西歌)   2023-10-09 16:02:00
原po意思是沒有release 分支吧master 當release 分支,卻被leader當個人分支用
作者: JoaoWang   2023-10-09 16:25:00
快逃 幾千個檔案有問題會看到死
作者: bachelorwhc (單身老王)   2023-10-09 16:37:00
笑死 剛講完繼續槓
作者: tyantmf (weekwnd)   2023-10-09 17:48:00
哪間
作者: gino0717 (gino0717)   2023-10-09 17:49:00
南無阿彌陀佛
作者: superpandal   2023-10-09 20:04:00
樓主什麼時候提到release... 這確實也只是個名稱 主管卡著master分支也可以理解 畢竟他要管理 但自己改的code不能動還常常不講改了什麼 然後別人的就駡的狗血淋頭 當然是雙標問題 其它什麼分支問題最好是下屬想淌這渾水 看來樓上好多主管 倉庫要完美自己好做事當然好一點是好的 但為了省自己成本轉移成本到下屬很糟糕Dr說的確實這社會真的好多沒管理能力還要擺譜的人
作者: peter98 (新兵)   2023-10-09 20:38:00
DrTech是閱讀能力有問題? 就說了,如果master不重要,就沒人會理他,既然有人抱怨了,那就是master有人在用。還在扯master不重要的是?唯一的例外就是原po自己天兵,也就是master是真的沒人在用,然後原po菜鳥不知道上來抱怨master被改。就算要亂測試一些東西,多數正常人也不會拿master來玩還是你想嘴有些project本來就不是master而是用main/mainline? oh man, 這麼計較名字的而不在意master要表達的本質? 我覺得也是滿厲害的,想必閣下溝通一定是字字必較。阿不對,就算proj是用main/mainline也不該蠢到上不能動的code到master
作者: superpandal   2023-10-09 21:00:00
master有人在用 怎麼會沒人在用 上到不能動的不是只有下面的人 至於覺得重不重要當然是心證 也確實沒標準 沒有四海皆通的標準
作者: labbat (labbat)   2023-10-09 21:40:00
以前沒有CI/CD 沒有版本管理系統還不是照樣coding(倒然後現在也是
作者: showshowman (show)   2023-10-09 23:13:00
主管應該是從cvs/svn時代沒實力隨年紀被推升上去的
作者: mepowerlmay (用心,找對人)   2023-10-10 01:01:00
Svn 就好 有這麼困難嗎
作者: WWIII (東邪西毒)   2023-10-10 02:01:00
快逃
作者: Suleika (Suleika)   2023-10-10 02:45:00
怎麼可能是這樣,快逃或成為切割捧懶人自己選一個會被檢討當然是超重要,這時還在糾結分支名稱別鬧惹
作者: sachung28 (00)   2023-10-10 05:21:00
不尊重人 才是問題反應意見被罵 請病假被強迫無償工作 這怎麼愉快玩耍看標題問madter 看內文感覺真賭爛的是待人方式 文不對題4ni
作者: pp129908 (阿兔)   2023-10-10 06:33:00
有這種北七leader也是很讚
作者: s678131 (Mu)   2023-10-10 09:53:00
快逃啊
作者: superpandal   2023-10-10 10:01:00
用什麼版控都不是重點 亂管是一定會亂 svn都可以用hook script弄ci/cd 差別在於git現在比較流行有現成
作者: ckp4131025 (ckp4131025)   2023-10-10 12:03:00
進code最大的權限在主管那裡的話,他自己進不覺得有什麼問題,因為責任本來就他全扛,有問題的是進爛code
作者: alan3100 (BOSS)   2023-10-10 16:46:00
除非刻意避開master字眼 不然哪個使用情境master不重要.走類gitlab切release branch也不會亂搞master
作者: lovdkkkk (dk)   2023-10-10 18:11:00
說不定是故意的這樣才容易出問題要加班處理多領加班費
作者: yoyoyolin (吳志和)   2023-10-10 19:33:00
網通寫軟體的很多嘴砲仔啊
作者: e12518166339 (耐綸)   2023-10-10 20:48:00
還有某一線ic設計大廠用rar版控,不用太絕望
作者: leoone (里歐一代)   2023-10-10 23:37:00
commit到master沒什麼錯 現在也有新流派在推廣 只是這是要有很多前提才能這麼幹的…..
作者: disabledman (m.m.)   2023-10-11 07:08:00
不正常,帶頭的壞了
作者: jobintan (Robin Artemstein)   2023-10-11 08:00:00
樓主直接貼公司的全名讓大家避避雷也算是功德一件呢……
作者: tsaigi (菜雞)   2023-10-11 12:59:00
這什麼恐怖故事XD
作者: superpandal   2023-10-11 20:51:00
rar...浪費空間和開發效率0
作者: ghicghic (為愛而生)   2023-10-11 21:19:00
這有什麼 我待過的外商 leader連local 和remote分枝都分不清楚
作者: DrTech (竹科管理處網軍研發人員)   2023-10-12 07:04:00
peter98閱讀能力與邏輯能力才有問題。有人在用master分支,與master分支是不是release分支有什麼關系不懂git的人別來嘴別人啦。git 從來就沒有標準在規定某個分支不能暫存開發程式碼。即使master或main分支,也絕對可以暫存開發程式碼。因為該分支不一定會Release。事實就是如此。歡迎提出合理證據討論。git也沒規定,兩人開發同一分支,不能暫存自己程式碼。本來就可以。尤其是該分支的所有者不是你,你根本沒資格管別人要存什麼程式碼。這才是git 設計的精神。你可以任意管別人開的分支,才是不合理的。master有別人用,有你用又怎麼樣?你不是所有者,你根本沒資格管別人怎麼管理master程式碼。我絕對認同該主管有問題,快逃,但問題不在使用git的方式。以上別斷章取義在戰別人了。
作者: ssccg (23)   2023-10-12 15:48:00
原PO只寫到原PO自己和主管有用,沒講到同事有沒有抱怨說不定master真的是那主管專用的
作者: newking761 (J三小)   2023-10-12 21:46:00
在垃圾公司的通常都是….你行就是你當主管了,為什麼你還是……
作者: ssccg (23)   2023-10-13 04:35:00
所謂的release用,有到master上的每個commit?這回應還是沒講到release(給客戶/下個phase),沒CICD的話很多用板控真的只是當有log的FTP在用啊
作者: BoXeX (心愛騎士團異端審判騎士)   2023-10-13 08:35:00
當FTP在用 然後其他員工被搞得很麻煩 離職真的比較快
作者: Litfal (Litfal)   2023-10-13 15:38:00
其實master就只是一個branch,沒什麼了不起的不過你應該是想抱怨他愛release就release,沒有任何的管控,但這是制度問題你可以跳出來「老闆你別寫扣了,你負責review就好」然後把他技術面架空後,再導入機制
作者: alsk1566 (Alsk )   2023-10-13 21:52:00
遇到這種你還不快跑
作者: jiannan1828 (我也不願意)   2023-10-13 23:47:00
南港某耳機公司裡面也很多SW leader這樣自己不會 叫RD教 然後聽不懂 還罵RD 這種人也有
作者: penril0326   2023-10-15 02:30:00
叫他吃屎
作者: labbat (labbat)   2023-10-15 20:37:00
git clone是哪招,每一次打掉重練開新repo叫大家複製嗎
作者: alan3100 (BOSS)   2023-10-17 03:14:00
主流git管理就那幾套各有試用環境 打迷糊仗說想怎樣就怎樣乾脆自己接案就好 愛怎樣管都憑個人喜好
作者: z1976 (z1976)   2023-10-19 15:35:00
你放心,我公司RD沒人知道GIT是什麼,就連IT也沒聽過CI/CD不過我是IC業
作者: loadingN (sarsaparilla)   2023-10-22 11:37:00
對啦 用perforce很屌
作者: salinia (as)   2023-10-22 12:19:00
沒 dev 嗎 我們 master 是要做 cicd 到測試機
作者: luluking (luluking)   2023-10-24 22:57:00
啥大廠,說一下,看看是不是真的大廠
作者: bobokeke (韭菜們安安)   2023-10-30 00:53:00
笑死,我最愛發PR,恨不得部門開讀書會來好好導讀一下我的風騷代碼

Links booklink

Contact Us: admin [ a t ] ucptt.com