123
返回列表 发新帖
楼主: 庞克莱门汀
跳转到指定楼层
上一主题 下一主题
收起左侧

[职场感言] 新人PR一次1200+, 400-这个算多吗

 
地里匿名用户
🔗
匿名用户-NPZZL  | 添加认证 | 2023-9-7 12:53:45 来自APP
我闭着眼给你直接approve  
回复

使用道具 举报

🔗
serendipity 2023-9-7 13:37:07 | 只看该作者
全局:
每个小功能用一个commit;  refactor的话单独弄一个别混淆;  refactor大的话弄几个。
回复

使用道具 举报

🔗
ny711 2023-9-7 16:06:13 | 只看该作者
全局:
感谢这个帖子,之前完全没意识到这个问题……也感恩同事没有骂我
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-SYYGH  | 添加认证 | 2023-9-7 21:22:40
你这个明显没做好计划,最好一小部分一小部分的修改,这样有错也容易revert找到问题所在。你上千行代码修改,万一有问题,revert完也很找到关键问题出在哪
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-IUP8G  | 添加认证 | 2023-9-8 07:08:26 来自APP
反正不拆开的话我是不会看的,太长了,就算都是name change也最好拆开分几部分不然容易有失误不好发现(只靠automation testing是不完全靠谱的
回复

使用道具 举报

🔗
 楼主| 庞克莱门汀 2023-9-8 22:06:15 | 只看该作者
全局:
匿名用户 发表于 2023-9-7 08:22
.1point3acres你这个明显没做好计划,最好一小部分一小部分的修改,这样有错也容易revert找到问题所在。你上千行代码修改 ...
. check 1point3acres for more.
好的学习到了,非常感谢!
回复

使用道具 举报

🔗
 楼主| 庞克莱门汀 2023-9-8 22:06:54 | 只看该作者
全局:
匿名用户 发表于 2023-9-7 18:08
反正不拆开的话我是不会看的,太长了,就算都是name change也最好拆开分几部分不然容易有失误不好发现(只 ...
.--
非常感谢,学到了,后来TL也确实和我说了内容太多了
回复

使用道具 举报

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

本版积分规则

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