[讨论] 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