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

[职场感言] Code review故意找一个容易过的人看?

   
🔗
sheephead 2022-4-21 12:19:35 | 只看该作者
全局:
贵司如果用github, 可以把service设成必须有service owner approval才能check-in
回复

使用道具 举报

🔗
adddaa 2022-4-21 12:20:33 | 只看该作者
全局:
adddaa 发表于 2022-4-20 15:48
真的,你但凡做个人,别人至于绕开你吗?同样的hiring bar进来的,凭啥人家就能过,到你这里就是提出一堆问 ...

你把你提的comment截几个图发上来呗,敏感信息可以打码。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-ZNR69  | 添加认证 | 2022-4-21 12:21:01 来自APP
pancakezzz 发表于 2022-04-20 20:32:21
楼主作为service owner要确保service code quality也是正常的,有些时候就算别人code review没加你,你也是可以jump onto,把自己设成watching或者必
你的这个从test出发考虑的思路挺好!不过我就是觉得你每次主动去看,人家还故意不加,就很膈应吧
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-ZNR69  | 添加认证 | 2022-4-21 12:46:29 来自APP
adddaa 发表于 2022-04-20 21:20:33
你把你提的comment截几个图发上来呗,敏感信息可以打码。
我为什么要发上来呢?我需要给你看吗?你是谁呀
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-ZNR69  | 添加认证 | 2022-4-21 12:51:35 来自APP
adddaa 发表于 2022-04-20 21:20:33
你把你提的comment截几个图发上来呗,敏感信息可以打码。
这么多人回复,就你戾气最重,你知道什么了你就说别人不做个人?不想骂你。你爱怎么review code那是你的事,我是来寻求经验,不是你这种一上来张嘴就喷的喷子。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-5Q185  | 添加认证 | 2022-4-21 13:02:12
看来很多人都有被reviewer挑刺的PTSD,哈哈哈。。。. 1point 3acres
楼主这情况应该技术上就可以解决,一般都可以设置必须owner的approval才能算signed off。
如果技术上都想尽办法了,那就从BQ找答案吧,1:1就不错,其实应该是junior来找你1:1解决的,但多半对方没什么工作经验,躲着这些问题。 你就该好好引导了。
. 1point3acres.com
至于comments合理不合理,我建议楼主这么反思一下, 你的comments的论点论据完不完整?有没有成文的规定或者业界标准来支撑?如果多数时候都只是你的opinion,那就说明你这有点问题了。
回复

使用道具 举报

🔗
adddaa 2022-4-21 13:12:32 | 只看该作者
全局:
匿名者 发表于 2022-4-21 00:46. check 1point3acres for more.
我为什么要发上来呢?我需要给你看吗?你是谁呀

寻求经验我让你发个review history有问题吗?凭什么啥信息不说,就一口咬定是别人的问题而不是自己吹毛求疵?就你这个态度,可以想象和你工作绝对不是什么愉快的体验。
回复

使用道具 举报

🔗
jstv 2022-4-21 13:30:58 | 只看该作者
全局:

有些人不过是小小的 owner 还真的以为自己是个谁,所有的 code 都要写成一个样,不一样就这个 comments 那个要改,其实完全没有必要。
.--
见树不见林,就是说的这种人。不自己反省,还怪别人不找你 review。
回复

使用道具 举报

全局:
Due近 时间紧 你跟我讲这两个声明要换一下顺序 提高readability?我可qnmd吧
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-LC1JM  | 添加认证 | 2022-4-21 13:55:38 来自APP
让我想起我们senior经常两个comment最后得用句号结尾也要comment一下
回复

使用道具 举报

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

本版积分规则

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