楼主: 匿名
跳转到指定楼层
上一主题 下一主题
收起左侧

[同事协作] 遇上一个对Code Quality 超级执着的组员,快被逼疯了,经验

   
全局:
不是Ctrl alt L 就解决code quality了吗🐶
回复

使用道具 举报

🔗
西瓜地 2023-3-18 07:35:21 | 只看该作者
全局:
知道为啥发明Go了吧,unused全给你报错,全自动给你format,这些小东西再也不用吵了。
回复

使用道具 举报

🔗
西瓜地 2023-3-18 07:39:28 | 只看该作者
全局:
顺手小改动非要开个新PR,结局就是没人改,前人写的typo关我什么事还要多浪费几分钟?
回复

使用道具 举报

🔗
西瓜地 2023-3-18 07:41:30 | 只看该作者
全局:
换一个mindset,"always leave it better than you found it", 像typo,unused, doc都属于这类。叽叽歪歪的人最烦了
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-BUHUO  | 添加认证 | 2023-3-18 08:15:23
看回复里有人说跟这样的人能学到很多东西的, 我只能说还有点学生气. 怕就怕你这还在学的酣畅淋漓呢, 老板哪天把你放 PIP 了, 因为在外人看来你就是 deliver 慢, PR问题多, 能力不行.
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-826JZ  | 添加认证 | 2023-3-18 08:39:34
产品组还是infra组?infra组代码质量确实可以扣一扣,产品组就完全没有必要了。如果你是产品组,建议跟manager反映一下,产品组就是快猛糙,在屎山里边雕琢没有意义
回复

使用道具 举报

🔗
kuerii 2023-3-18 08:42:52 | 只看该作者
全局:
挺正常,磨合一阵子就好了,都是这么过来的
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-R47BZ  | 添加认证 | 2023-3-18 08:43:06
看样子他是喜欢小PR。那你就发小的。一个大任务拆开,这样review快,然后也凸显自己的工作量。反正我自己喜欢发小PR,也反感别人发大PR,大PR花别人时间长,很多东西也看不出为啥要改。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-IMQOQ  | 添加认证 | 2023-3-18 08:52:17 来自APP
匿名用户 发表于 2023-03-17 12:01:01
你没遇到过一定要把两个单词的名字改成一句话的。。。在java语法里
class A = new class(). ----
这么一个statement能写三行。。。
现在不是清一色点build么
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-HPYDV  | 添加认证 | 2023-3-18 08:52:44
同遇到,比我高一级,直接跟他骂起来了,老板过来调节,后来他老实了不少,最多2,3个了,之前30+,只有来硬的才好使,拿出同归于尽的魄力
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容
  • ☑ PIP / DevList/ Need Support 等话题也已开设 🔗专版

本版积分规则

>
快速回复 返回顶部 返回列表