查看: 1990| 回复: 3
跳转到指定楼层
上一主题 下一主题
收起左侧

[自我提升] 工作两年的engineer被要求code review组里70%的PR

🔗
匿名用户-UYZ1E  | 添加认证 | 2022-7-10 00:46:50 |倒序浏览

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
本帖最后由 匿名 于 2022-7-9 09:26 编辑

楼主最近被manager警告表现的不好,其中有一个原因就是不怎么review其他engineer的代码。楼主因为水平比较低,就只review那些比较容易理解的PR,大概占总数量的25%吧。manager问楼主为什么不愿意review别人的代码,楼主说很多PR不能完全看懂,所以常常有不确定的情况,就不去approve。
manager说,那就在其他人的PR里面提问,让自己了解那些不懂的地方。令人震惊的是,manager说,那些看不懂的PR,可以直接approve。
楼主辩解说,如果两个人都不懂一个PR,却都直接approve了这个PR,那么这个PR甚至可以直接merge了,有bug怎么办?
manager说,那是另外一个问题,你不用管。
.--
请问,manager的这套说法合理吗?他多次说过类似的话。楼主没在别的公司待过,不清楚别的公司都是怎么看待谁应该review,以及要review多大比例的PR。
组里的人员组成情况:1.5个senior(有一个senior很多时间在别的组工作,不是full time在我们组) + 2个L4 + 2个L3.--

回复加米!谢谢!. .и



上一篇:买它pip是直接走人吗?
下一篇:我就问一句, meta会撕offer吗?
全局:
工作两年还review不了PR,那perf确实有点问题
回复

使用道具 举报

全局:
老板说的有些道理,review PR确实是学习的好方法,不过确实是有点花时间的。
另外,如果看不懂的PR直接给approve我倒是不赞同。我觉得可以让提交PR的人把Description写得好一些,给一些background/context,这样可以极大帮助reviewer看懂code。

评分

参与人数 1大米 +1 收起 理由
zzupupup + 1 给你点个赞!

查看全部评分

回复

使用道具 举报

地里匿名用户
🔗
匿名用户-APHKF  | 添加认证 | 2022-7-10 00:55:16 来自APP
Manager在找茬,只是想找理由证明LZ的performance不好。PR相对来说是一个比较容易找的地方。一般来说除非组里经常有很长时间没人review不能close的PR,这方面不会有人注意

评分

参与人数 1大米 +1 收起 理由
zzupupup + 1 给你点个赞!

查看全部评分

回复

使用道具 举报

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

本版积分规则

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