顯示廣告
隱藏 ✕
※ 本文轉寄自 ptt.cc 更新時間: 2023-10-09 19:28:01
看板 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
andy00andy: 這種能做到leader,還不逃?1F 10/08 03:11
Apache: 看起來你們沒有code review系統 也沒有CICD2F 10/08 03:41
Apache: 雖然有導入可能也會沒開設定 還是可以自己進code
jonathan793: 塊陶4F 10/08 03:57
Firstshadow: 這該不會也是你leader的第一份吧5F 10/08 04:08
s860134: 切記 業界沒有這樣6F 10/08 04:19
s860134: 正常公司不會這樣 這種只能靠你自立自強
s860134: 外商也有這種 程度沒到大學畢業的
IhateOGC: 有些人單純嘴砲能力上位的9F 10/08 04:51
IhateOGC: 勸你還是快閃
Apache: 不懂技術不一定是問題 但不懂卻碰技術就是問題11F 10/08 04:54
vongola12324: 奇文觀賞12F 10/08 05:25
vongola12324: 每個業界都這樣幹那還得了
vongola12324: 平衡推
vongola12324: 順便偷問一下是哪一間
zebra10029: 快逃16F 10/08 06:32
buyfood: 我碰過類似的,做三個月果斷烙跑17F 10/08 07:35
yamakazi: Jenkins gerrit 這種免費的CICD自己串一串不會?這種組18F 10/08 07:41
yamakazi: 內自己就可以搞定的,覺得看不慣就自己建流程是多難?
airtsubasa: 製造業正常啦20F 10/08 07:44
yamakazi: 全部都要等到公司準備好,那事情就不用做了21F 10/08 07:44
hsuchengmath: 建一個integration test 讓你主管不能推有問題的扣22F 10/08 07:49
hsuchengmath: 上去啊
peter98: 我認真回答你~你主管滿鳥的~他在科技業主管群裡的pr大概24F 10/08 08:03
peter98: 只有25
NDark: 拉出一個dev/release, 放棄已經存在的master讓他玩沙26F 10/08 08:32
luckid: 好奇是哪一間27F 10/08 08:46
loadingN: 不是業界 是三流系統廠28F 10/08 08:58
sniper2824: 這樣pr還有25喔29F 10/08 09:01
qwe70302: 有些leader根本連code都不會寫,確實有pr2530F 10/08 09:36
wulouise: 嗯,一個字跑31F 10/08 09:53
peter98: 有  一定有更爛的主管幫原文說的這位稱到25 XD32F 10/08 10:02
peter98: 我之前隔壁組~空降一個鳥主管到今年初2年半~底下的人換了
peter98: 一輪半~最後他的主管職位被他老闆拔掉~最後兩位從他手底
peter98: 轉組的人跟主管的主管說: 爛死了這甚麼主管?怎麼選的?
peter98: 那你說原文這位老闆有沒有PR25?
lturtsamuel: 沒有code review就算了,跑不動的代碼竟然能進到mast37F 10/08 10:29
lturtsamuel: er?世界奇觀
sniper2824: 真可怕= =39F 10/08 10:38
justaID: 我覺得塊陶 一個leader沒有開發制度也沒有善待下屬的氣40F 10/08 10:44
justaID: 度 在他底下只是浪費時間 不健康的成長  除非你能上達天
justaID: 聽 否則更上層的老闆只會信他不會信你 捅包你要背鍋 就
justaID: 算你做得很好 leader也只會收割你的功績
justaID: 一個不適任的leader能上位可以有100種時光背景原因 一旦
justaID: 上位成功 再爛也沒那麼容易被拉下 尤其台廠很看重面子
justaID: 所以我覺得只有塊陶一條路
justaID: *時空背景原因
luke72: 網通廠這樣的leader很正常,因為程度正常的早跑了48F 10/08 11:32
johnbill: 快逃49F 10/08 11:52
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
luke72: 懂技術的人都因為政治不行而被鬥去更高薪的產業了
magic83v: 還以為是我發的 趕快level up 趕快跳槽才是真的56F 10/08 12:30
WaterLengend: 遇過類似的,結果看那主管的領英越混越差,可悲57F 10/08 12:46
ppc: 公司問題,建議換工作。58F 10/08 15:02
iceman5566: 紅明顯 台灣某新創獨角獸的 IT head 超常假日推 cod59F 10/08 15:05
iceman5566: e 到 master,然後也不講,超靠北,該 head 是一個老
iceman5566: 人,聽說是靠關係空降的,只能說這種文化並不是偶見
mike8469: 快逃R62F 10/08 15:14
mike8469: 大學畢業的新人都不會這樣幹.....
nayeonmywife: Wow64F 10/08 15:18
xsdferty035: 不能跑還進==65F 10/08 15:21
realbout: 這leader是不是台大66F 10/08 15:22
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: 樓上說的是 混蛋其實蠻多的 這些人為了自己的生存可72F 10/08 17:03
superpandal: 以無所不用其極 其實就是刻意亂凸顯你的錯誤和缺點
superpandal: 這波操作下沒有問題都會變有問題 打訊息差逼你遷就他
superpandal: 說回分支問題 多數人都是一人專案才直接用master 當
v86861062: 真慘76F 10/08 17:17
superpandal: 然不一定是這種狀況77F 10/08 17:17
superpandal: 我都是直接當ftp用 畢竟管理層的不雞掰的很少 被惡意
superpandal: 針對還要替他們管理著想就太奴了 如果是好人當然就弄
superpandal: 到好
DrTech: 很爛沒錯。但片面之言看看就好。如果沒上產品,你master怎81F 10/08 17:31
DrTech: 麼改,其實都沒差。
DrTech: 真正有上產品,通常最後程式碼也未必在master分支。很多事
DrTech: 情看表面沒意義。
DrTech: 最後程式碼的owner是誰?需要品質嗎? 如果owner不是你,
DrTech: 也不需要品質,那你沒權限。
DrTech: 真的別把master分支當作多重要的一件事情,玩久了你就知道
DrTech: 了,死守絕對的標準,還不如變通一下,另外拉其他分支自己
DrTech: 玩就好。Release絕對沒有規定要在master才叫專業。
loadingN: 對啦,不是release的,沒什麼關係,但既然沒review就在90F 10/08 17:43
loadingN: local自己玩啊,推上去有意義嗎...
superpandal: 至於業界是不是都這樣 我只能說沒見過倉庫是很整齊乾92F 10/08 17:45
superpandal: 凈的 或許誤commit 或許有疙瘩的commit
superpandal: 開源倉庫都不一定乾凈了 更別說職場混亂狀況
DrTech: 我待過標準規則一堆,做錯了要扣薪水,完整乾淨的倉庫。但95F 10/08 17:51
DrTech: 我不認同管理repository有一個適用於所有公司部門的標準。
DrTech:  存在即是合理,程式碼或repository爛,還能繼續領薪水,
DrTech: 代表該單位的獲利價值,或專業根本不在此而已。
DrTech: 專業當然很重要。但不專業的人竟然可以混得更好。通常是,
DrTech: 你認為的專業,跟別人認為的專業不同而已。
DrTech: 別人又不靠乾淨的master分支賺錢時,這件事情就真不重要了
DrTech: 。
superpandal: 應該沒公司靠乾凈軟體倉庫賺錢 會這樣要求的多半是管103F 10/08 18:01
superpandal: 理職身在其位但想節省自己的心力搞的 但現實不可能
airtsubasa: 這不就跟變數命名 檔案命名一樣,誰是老大105F 10/08 18:03
superpandal: 節省心力有更好的方法就是 git不萬能 無法解決不會管106F 10/08 18:06
superpandal: 理的人的管理問題
typeking: 想也知道不可能業界都這樣,你還會專程上來發文問就知道108F 10/08 19:30
typeking: 你判斷能力也有待加強
typeking: 繼續待在這公司就好
yamagishi: 問題是主管雙標吧111F 10/08 19:32
Csongs: 嗯嗯112F 10/08 19:42
fu3mo6: 把default branch 改成 main113F 10/08 19:45
Apache: 確實 master不夠inclusive114F 10/08 19:59
f26724309: 這也太鬼XD 哪家阿讓大家笑笑115F 10/08 20:06
kimisme9386: 快逃啊116F 10/08 20:14
feathergod: 會這樣做事 就是不靠程式碼賺錢 反正寫得爛 沒有規矩117F 10/08 20:47
feathergod:  公司照樣賺 薪水照樣領 幹嘛認真弄
gpctv: 這真的還好,我還聽過某金融連branch 切都沒切的,還說公司119F 10/08 20:52
gpctv: 專案很小,這個不需要,但....是公司的核心系統,版控不曉
gpctv: 得在做三小的
knives: 有這種主管,就算有review機制也是枉然好嗎122F 10/08 21:03
blazers08: 快逃123F 10/08 21:37
abc0922001: 有版控總比沒版控好出事至少能追到犯人XDD124F 10/08 21:47
bearkevin: 不逃?125F 10/08 21:56
N91: 逃吧 沒能力的主管126F 10/08 22:00
jay123peter: 這種沒同理心主官都很雷...127F 10/08 22:07
drakd4d: 夭壽128F 10/08 22:24
viper9709: 跑不動的代碼進master www129F 10/08 22:38
Mike1109: 心臟真大顆130F 10/08 22:45
ZakuSIN: 往好處想 "至少有版控"131F 10/09 00:26
ZakuSIN: 不爽不要做
peter98: 反推那些說master不重要的,如果master不重要,那就代表133F 10/09 00:34
peter98: 他沒再用,這樣應該沒人鳥master才對,如果有人鳥,代表
peter98: master還是有在用,這樣就不應該上動都不能動的code去
peter98: master,另外說的release,額,你們又知道release不會從
peter98: master sync過去了嗎? 一堆歪理。
peter98: 每個team開發流程確實不一樣,如果master真的沒人鳥,那
peter98: 原po抱怨master的事情幹嘛? 阿不就是有人用嗎? 邏輯不好
peter98: 工作寫程式或當經理管理應該也都不怎麼滴
peter98: 除非原po他自己不知道master其實沒人在鳥~這樣要嘴原po我
peter98: 覺得就說得過去。
superpandal: 有人說到重點 問題是主管雙標143F 10/09 10:31
superpandal: master當然有人在用阿 主管卡著用阿 這本來就是雙標
superpandal: 問題 repo管的好不好是另外一回事
superpandal: 不是分支管理不重要 是很多公司來講看起來不重要 而
superpandal: 且推commit本身就有機會有事故或習慣不同 要完美的倉
superpandal: 庫狀態是不可能的 工具可以輔助管理 但無法避免管理
superpandal: 還可以扯到寫程式如何如何 你沒上帝視角看不了這麼多
superpandal: 即便人在你面前都不一定可以看的出來
superpandal: 有寫輪眼的人可能可以看出來一點點 但沒這麼強大
hakosaki: 快跑....152F 10/09 10:57
jsa1090: 有CI/CD的話 可以設定基本lint failed卡住不能merge 或是153F 10/09 10:58
jsa1090: github直接設定沒有review不能merge
jsa1090: 如果主管只是懶惰的話 寫個script讓他只要一行指令就可以
jsa1090: git pull rebase from master and checkout new branch
superpandal: 是 但不可避免中間會有人為因素 你整條流水線因素都157F 10/09 11:03
superpandal: cover到才稍微可以 但未知總是災難之源
B0988698088: 講master不重要的是在共三小 即使可以靈活運用也不159F 10/09 11:49
B0988698088: 是像這樣被老害亂用好嗎 抬三小槓
bachelorwhc: 這板槓精真多161F 10/09 13:41
DrTech: master本來就不重要。搞不好人家是用main啊,搞不好人家re162F 10/09 15:16
DrTech: lease不在 master啊。何必執著於名稱?有什麼國際標準,或
DrTech: 專業論文說master比較好? 玩得通透的人,本來就不是看分
DrTech: 支名稱決定品質與流程好壞。
DrTech: 而且如果master 沒有 release 作用,  為什麼master分支不
DrTech: 能被當成程式碼暫存用途? 那個國際標準或專家說master分
DrTech: 支不能用來暫存程式碼的? 歡迎提出證據與事實啊。
DrTech: master本來就可以當成個人開發用的暫時分支。git本來就是
DrTech: 這樣設計的。至於master分支,要不要當release分支,根本
DrTech: 沒標準過。
leonEEman: 笑死 這leader是剛畢業嗎172F 10/09 15:44
Csongs: 原po意思是沒有release 分支吧173F 10/09 16:02
Csongs: master 當release 分支,卻被leader當個人分支用
JoaoWang: 快逃 幾千個檔案有問題會看到死175F 10/09 16:25
bachelorwhc: 笑死 剛講完繼續槓176F 10/09 16:37
tyantmf: 哪間177F 10/09 17:48
gino0717: 南無阿彌陀佛178F 10/09 17:49

--
※ 看板: Soft_Job 文章推薦值: 0 目前人氣: 0 累積人氣: 47 
分享網址: 複製 已複製
guest
x)推文 r)回覆 e)編輯 d)刪除 M)收藏 ^x)轉錄 同主題: =)首篇 [)上篇 ])下篇