前端PR合并不应随意点击,而需明确准入门槛:PR描述须含Related Issue、Impact Scope、Testing Done;CI检查(ESLint、Jest覆盖率、Prettier、size-limit)必须通过;自动分配Reviewer并至少2人审核;合并前验证main分支构建及staging环境核心链路。

前端团队的 PR 合并不该靠“谁有空谁点一下”,而是要有明确的准入门槛和可执行的判断依据。否则,npm run build 成功、git push 完成,不等于代码能进 main 分支。
PR 描述必须包含哪些字段才能被受理
很多 PR 被卡在第一步——审查者根本不知道这改的是啥。描述不是越长越好,而是要填够关键信息才能触发自动化检查和人工评审。
-
Related Issue:必须关联#123类型的 issue 编号,没有就拒绝受理 -
Impact Scope:明确写清影响范围,比如“仅影响/user/profile页面的表单校验逻辑”,而不是“修复了用户模块” -
Testing Done:列出实际执行过的验证项,例如“✅ 手动测试 Chrome/Firefox/Safari;✅ 新增ProfileForm.test.tsx覆盖 3 个边界 case” - 截图或录屏非强制,但涉及 UI 变更时,没图的 PR 审查者有权要求补交
哪些检查项必须通过才能合并
CI 流水线不是摆设。前端项目常见陷阱是本地跑通就提 PR,结果 CI 报 ESLint: 12 errors 或 jest: coverage below 80%,来回重推浪费所有人时间。
-
eslint --fix必须零 error,warning 不阻断但需在 PR 描述中说明原因(如“暂未迁移旧组件,已标记// eslint-disable-next-line”) -
jest --coverage要求变更文件的单元测试覆盖率 ≥ 80%,新功能类文件必须 ≥ 90% -
prettier --check检查格式一致性,尤其注意.tsx文件中 JSX 属性换行和括号对齐 - 构建产物体积变化超过 ±5KB 时,
size-limit会失败,需在 PR 中解释增量来源(如引入了lodash/debounce)
谁该被自动分配为 Reviewer
把 PR 甩给“随便谁看看”等于没人看。Gitee 和 GitHub 都支持基于路径的 reviewer 自动分配,但配置不当反而造成漏审。
把任意文章或文档内容转换成精美的全屏翻页 HTML 演示文件(幻灯片风格)。当用户发来一篇文章、文档链接、Markdown 内容、或粘贴的文本,并且希望生成演示用的 HTML 时触发。支持任意内容来源:iWiki 链接、KM 链接、本地文件路径、直接粘贴的文本。生成的 HTML 文件保存到本地工作区,并通过企微直接发送给用户。触发词示例:帮我把这篇文章做成PPT、转成演示文档、生成HTML幻灯片、做个演示、转成可以演示的格式、article to deck、make a slide。
- 按目录路由:修改
/src/components/Button/就自动 @ 组件库维护者;改/src/pages/Dashboard/就 @ 对应业务线前端 - 避免跨域指派:不要让负责支付模块的人审登录页的 CSS 变更,除非改动涉及公共样式层(如
/src/styles/variables.css) - 每个 PR 至少需 2 名 reviewer,其中一人必须是模块 owner(由
CODEOWNERS文件定义),另一人可以是同组成员 - 如果 PR 修改了
package.json的依赖,必须包含security audit结果,且由 DevOps 或安全负责人额外确认
合并前最后一步常被跳过的检查
很多人以为点完 “Merge” 就结束了,但真正出问题的往往在合并后一秒——比如 main 分支上立即出现 build failed,或者某个页面白屏。
- 确认 CI 最终状态是绿色,且是针对
main上最新 commit 的结果,不是基于你 push 时的旧 base - 手动验证一次
git checkout main && git pull && npm ci && npm run build,确保没漏掉.env或public/下的静态资源变更 - 如果 PR 引入了新的环境变量(如
REACT_APP_FEATURE_FLAG),必须同步更新.env.example并在描述中注明 - 合并后 10 分钟内,至少一人要在 staging 环境走一遍核心链路,截图发到团队群——这不是形式主义,是防止打包产物漏文件的最简防线
最难的不是定规则,而是让每条规则都有对应的落地钩子:模板强制填写、CI 拒绝低覆盖率、CODEOWNERS 精确到目录、合并后有人盯第一波日志。缺一环,规范就变成文档里的一段文字。
前端入门到VUE实战笔记:立即使用
在学习笔记中,你将探索 前端 的入门与实战技巧!










