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

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

   
全局:
公司的 Coding standard 做的很好,testing, doc, comments, release notes. 大部分都有规定好的,一个team代码风格应该是一个风格。 很多code review 要过几次才能养成良好的习惯
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-T1ZJP  | 添加认证 | 2024-9-11 03:48:48
匿名用户 发表于 2022-4-20 13:35
这是Interview BQ的问题!
一个Code review comments太多时, 就不应该继续加comments了. 这时你该跟作者sch ...

大佬,请问这个是senior的bq吗?
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-GYVQY  | 添加认证 | 2024-9-11 05:45:17
不知道“引起一些TT”什么意思。
但是,如果有问题的改动可以merge,并引发实际问题,而且“经常”发生,那是不是自动化测试有改进的空间?. 1point3acres.com
应该自动发现、自动block这样的改动,至少不应该“经常”出这样的问题。. ----


另一个角度,其他人也提到了,“没太大关系的人”做code review没问题,但只有他们的approval不应该允许改动被merge。或者,可以一开始允许,但“经常”出问题后,不应该继续允许。
回复

使用道具 举报

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

本版积分规则

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