周一上午九点,林晨坐在工位上,盯着 GitHub 页面上那个刚刚创建的 Pull Request(代码合并请求),心跳有些加速。这是他在新公司的第一个 PR。
过去一周,他几乎每天加班到晚上十点,周末也泡在代码里。任务不算复杂:为一个 DeFi(去中心化金融)项目的前端添加一个质押收益计算器组件。他用了自己最熟悉的 React 方式完成了开发,自测了几遍,功能都正常。
鼠标在“Create pull request”按钮上悬停了十几秒,林晨深吸一口气,点了下去。页面跳转,PR 编号 #347 出现在屏幕上。他顺手在 Slack 的技术频道里 @ 了 Tech Lead 陈峰:“陈哥,质押计算器组件的 PR 已提,麻烦有空 review 一下”。
消息刚发出去,陈峰几乎秒回:“收到,上午处理”。
林晨松了口气,起身去接水。路过陈峰的工位时,看到对方已经点开了 PR 页面,眉头微皱,手指在键盘上快速敲击。回到座位,林晨强迫自己不去刷新页面,转而开始看项目里另一个模块的代码。但注意力总是不集中,每隔几分钟就忍不住瞥一眼屏幕右下角的 Slack 图标。
这种等待代码审查的感觉,比他预想的要焦虑。在上一家跨境电商公司,他的代码审查通常很顺利——十年老员工,对业务系统和代码库了如指掌,提交的 PR 往往只有些格式小调整。但在这里,一切都是新的:新技术栈、新编码规范、新的最佳实践,甚至新的思维方式。
上午十点半, Slack 图标终于闪动起来。
陈峰:“@林晨 看完你的 PR 了,问题比较多,我们快速过一下?现在方便的话来小会议室”。
林晨心里一沉,回复:“好的,马上来”。
拿起笔记本和笔,他走向走廊尽头的小玻璃会议室。推门进去时,陈峰已经坐在里面,面前的 MacBook 屏幕上正是那个 PR 的页面,右侧评论栏里密密麻麻的红色标记。
“坐”。陈峰指了指对面的椅子,语气平静但严肃,“我们先整体说,然后你回去逐条修改”。
林晨坐下,打开笔记本准备记录。
“首先,功能实现没问题,计算逻辑是对的”。陈峰开门见山,“但问题出在代码质量、安全性和可维护性上。我一条条说,你记一下”。
他滚动页面到文件开头
(本章节未完结,点击下一页翻页继续阅读)