请
登录
后使用快捷导航
没有帐号?
注册账号
导读
Guide
解锁权限
功能中心
话题
求职
攻略
帮助
搜索
⌘
Ctrl
k
智能搜索
Google搜索
站内搜索
体验新版
权限解锁
登录
注册
首页
论坛
Offer多多
面试经验
Salarytics
Visa Tracker
交友
Instant
精品网课
生活攻略
手机APP
商务合作
<
登录
注册
升级
关注
扫码关注一亩三分地微信公众号
一亩三分地Warald
一亩三分地华人生活
一亩三分地求职移民
一亩三分地海外投资
帮助
TOP
置顶话题
最新回复
最新主题
最新热门
获取积分
新手指南
淘贴
我的订阅
我的帖子
VIP通行证
购买蓝莓
企业招聘广告
移民律师专栏
洽谈其他合作
道具[匿名/删除/提升/关闭/等]
考试
附件
勋章
许愿墙
去广告
面试辅导
全民竞猜
每日签到
每日答题
超级匿名
微信解绑
话题
学校
公司
地区
证券
求职辅导匹配
DS501 数据科学面试40+真题讲解
瞄准秋招:暑假跟着Shawn老师刷题!
DS601 AB Test从入门到精通
CS601 谷歌工程师讲系统设计
DS401 资深数据科学家教你破解DS面试
CS401 谷歌招聘委员会成员讲解如何准备面试
DS402 Resume Workshop
DS401A 模拟面试-数据科学
DS404B 模拟面试-机器学习
CS404A 模拟面试-算法
CS404B 模拟面试-系统设计
锦晖律师事务所H1B签证申请讲座
CS入门攻略
CS求职攻略
PM求职攻略
系统设计攻略
数据科学攻略
回国礼物礼品最全清单
如何在新城市快速省心安家
车盲如何购买二手车
收到医院的天价账单怎么办
更多
苹果应用商店
谷歌应用商店
百度网盘apk
商务合作
常见问题
新手上路
App下载
帖子
投票
活动
版块
群组
日志
用户
一亩三分地
»
论坛
›
职场达人
›
职场达人
›
工作两年的engineer被要求code review组里70%的PR
返回列表
查看:
1990
|
回复:
3
直达
[自我提升]
工作两年的engineer被要求code review组里70%的PR
职场达人
地里匿名用户
匿名卡
变色卡
关贴卡
🔗
匿名用户-UYZ1E
|
添加认证
|
2022-7-10 00:46:50
|
倒序浏览
本楼:
👍
0
0
👎
注册一亩三分地论坛,查看更多干货!
您需要
登录
才可以下载或查看附件。没有帐号?
注册账号
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
.--
回复加米!谢谢!
. .и
收藏
1
淘帖
0
好苗
0
杂草
0
上一篇:
买它pip是直接走人吗?
下一篇:
我就问一句, meta会撕offer吗?
相关帖子
一个好的 AI Engineer到底创造了什么价值?
感觉做了很多东西,但是老板不管升职怎么办?
加入早期初创公司(Seed-B轮)要重点注意的事情
人类学re面试
是继续做推广搜还是转agent
拿了身份后,做了5年的shi山工作,要立即quit吗?
有砖搬的都是侥幸啊被雷的都是人生
AI 大神买了7千万豪宅
140approve了想跳槽
现在找工还要不要刷题
亚麻reorg后升职前景
都 2026 年了,为啥找工作还要刷题?有感而发
心里太难受了,面试搞砸了之后晚上和父母通话结果被催着找工作
Stripe 裁员了吗
Designer 现在都自己写PR了吗
Qwen3.8-27B 支持1m上下文 benchmark benchmark吊打meta野狗模型
保加拿大绿卡,还是留美国工作。
帮朋友问,opt 最后一年, 被大厂裁员(不能internal),请问哪些公司可以收1年opt
整组Reorg到印度的director下面
回复
使用道具
举报
提升卡
置顶卡
关贴卡
开贴卡
变色卡
千斤顶
gdhgdjj
当前离线
主题
帖子
102
积分
不准访问
积分
102
大米
颗
鳄梨
个
水井
尺
蓝莓
颗
萝卜
根
小米
粒
学分
个
注册时间
2022-7-5
最后登录
1970-1-1
狗仔卡
推荐
gdhgdjj
2022-7-10 00:49:53
来自APP
|
只看该作者
本楼:
👍
3
0
👎
全局:
77%
23%
955
290
工作两年还review不了PR,那perf确实有点问题
回复
支持
3
反对
0
使用道具
举报
stupidBoy
当前离线
主题
帖子
278
积分
中级农民
积分
278
大米
颗
鳄梨
个
水井
尺
蓝莓
颗
萝卜
根
小米
粒
学分
个
注册时间
2011-3-22
最后登录
1970-1-1
狗仔卡
推荐
stupidBoy
2022-7-10 00:53:55
来自APP
|
只看该作者
本楼:
👍
1
0
👎
全局:
93%
7%
261
21
老板说的有些道理,review PR确实是学习的好方法,不过确实是有点花时间的。
另外,如果看不懂的PR直接给approve我倒是不赞同。我觉得可以让提交PR的人把Description写得好一些,给一些background/context,这样可以极大帮助reviewer看懂code。
评分
参与人数
1
大米
+1
收起
理由
zzupupup
+ 1
给你点个赞!
查看全部评分
Coderust: Hacking the Coding Interview
回复
支持
1
反对
0
使用道具
举报
地里匿名用户
🔗
匿名用户-APHKF
|
添加认证
|
2022-7-10 00:55:16
来自APP
本楼:
👍
0
0
👎
Manager在找茬,只是想找理由证明LZ的performance不好。PR相对来说是一个比较容易找的地方。一般来说除非组里经常有很长时间没人review不能close的PR,这方面不会有人注意
评分
参与人数
1
大米
+1
收起
理由
zzupupup
+ 1
给你点个赞!
查看全部评分
回复
支持
反对
使用道具
举报
返回列表
B
Color
Link
Quote
Code
您需要登录后才可以回帖
登录
|
注册账号
职场达人
↑ 本版用于讨论职场各种干货话题,闲聊请去🔗
聊聊
或者🔗
匿名版
❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
☑ 求职、面经等去 🔗
北美求职
和 🔗
回国求职
大区,刷题和学习请去 🔗
终身学习
大区
☑ 请去专版发布 🔗
内推
, 🔗
招聘
信息,和讨论 🔗
创业内容
☑ PIP / DevList/ Need Support 等话题也已开设 🔗
专版
本版积分规则
发表回复
回帖后跳转到最后一页
浏览过的版块
刷题
各校讨论历史存档
求职(非面经)
加入早期初创公司(Seed-B轮)要重点注意的事情
一个好的 AI Engineer到底创造了什么价值?
《硅谷超级家长课》总结
2026年Q3跳槽经验分享
谈谈美国relo加拿大的体验
2026 Q2跳槽总结
张爱玲读后感 《流言》
[分享/励志] 从F2到CS Master再到工作:致每一个在迷茫中深夜痛哭过的你
老房子漏水排查与省钱修管全记录| 500刀的教训
San Francisco field office 面试经验分享
>
论坛导航
快速回复
返回顶部
返回列表