团队扩大了,前端每天几十个 PR 全卡我这了😂 评审 + 合并 + 部署,大家的工作流是怎么搭的?

6 小时 15 分钟前
 qxiaoyuan

最近团队多了几个伙伴,大家产出很猛,每天几十个 PR 堆过来。评审、合并、部署现在基本都集中到我这一块,我成为影响大家效率的一环了哈哈。

我现在的做法:用 Codex 的 bot 在 PR 上自动评审,每个 PR 先让它过一遍、给意见,我再看它的结论决定要不要改、要不要合。这步确实省了不少力,但还有两个点卡着。

一、想做多 AI 评审,但不知道怎么编排

只靠单个模型的意见我不太放心,想让多个 AI(比如 Codex 、Claude 再加一个)一起评,交叉验证,减少单模型的漏判和误判。

真要这么干,几个问题:

有没有人真在跑多 AI 评审的,怎么编排的?

二、非团队成员的提交,Vercel 不给自动部署

我们前端挂在 Vercel,只要提交的作者不是 Vercel 团队成员,他的 PR / 合并就不会自动部署,每个都要我手动点一下才部署。

三个解决方法:

你们遇到这个 Vercel 授权是怎么处理的?

大家真实在用的 PR 自动评审 + 自动部署工作流,尤其多 AI 评审这块,很想取取经。

805 次点击
所在节点    问与答
6 条回复
davidoff567
5 小时 50 分钟前
互联网公司的话审批太细会拖累进度,像原有的 CodeReview 可以舍弃了( eg:这个方法为啥要这么写),出了事艾特对应的人能及时响应就行,用户当测试
zhangli2946
5 小时 41 分钟前
评审交给有技术上下文的人 (来背锅)
把自己从部署环节摘出来 (审核流程正确性,反复运行正确流程)
jackOff
5 小时 41 分钟前
我现在强制要求写好 git 提交记录,改了什么,怎么测都必须严格写好,然后让 ai 先自动化操作一遍测试一下看看和 git 提交记录是否对的上,然后再随机测测
yidinghe
5 小时 40 分钟前
不要给自己上强度。你的做法是每个 AI 都在给你上强度而不是减轻你的负担。

正确的做法是,你负责审设计,AI 负责审细节,先让 AI 总结设计方面的变更,然后你对照需求看是否可以接受。然后 AI 查细节,是否有逻辑遗漏或计算方面的问题,当然最后还是要测试的。
lizhenda
1 小时 31 分钟前
还用 Vercel ?公司这么有钱?除了最开始没出 AI 的时候省事,现在它还有什么优势?
lizhenda
1 小时 30 分钟前
其实我觉得你担心的点不太对,最大的可能是整个前端团队都被优化掉,或许非常突然。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1249409

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX