Pull Request 提交指南
本指南将帮助您了解如何正确地向 agentic-ui 项目提交 Pull Request (PR),包括从分支创建到合并的完整流程。
📋 目录
🚀 前置准备
1. Fork 项目
如果您是外部贡献者,请先 Fork 项目到您的 GitHub 账户:
2. 环境设置
🌳 分支管理策略
分支命名规范
创建开发分支
📝 提交规范
Commit Message 格式
我们使用 Conventional Commits 规范:
提交类型 (type)
- feat: 新功能
- fix: 错误修复
- docs: 文档更新
- style: 代码格式调整(不影响代码运行的变动)
- refactor: 重构(既不是新增功能,也不是修改bug的代码变动)
- perf: 性能优化
- test: 添加或修改测试
- chore: 构建过程或辅助工具的变动
示例
提交前检查清单
在提交代码前,请确保:
- 代码通过 lint 检查:
pnpm lint - 所有测试通过:
pnpm test - 类型检查通过:
pnpm tsc - 代码格式化:
pnpm prettier - 更新相关文档
- 添加或更新测试用例
🔄 创建 Pull Request
1. 推送代码
2. 在 GitHub 上创建 PR
- 访问您的 Fork 仓库页面
- 点击 "Compare & pull request" 按钮
- 选择正确的分支:
- base:
ant-design/agentic-ui的main分支 - compare: 您的功能分支
- base:
3. PR 标题和描述
PR 标题格式
PR 描述模板
4. 标签和里程碑
适当添加标签:
feature- 新功能bug- 错误修复documentation- 文档相关enhancement- 功能增强performance- 性能优化
👥 代码审查流程
审查前准备
确保您的 PR:
- 通过所有 CI 检查
- 代码覆盖率不降低
- 没有合并冲突
- 描述清晰完整
审查过程
- 自动检查: CI 会自动运行测试和检查
- 代码审查: 维护者会审查代码质量、设计和实现
- 反馈处理: 根据审查意见修改代码
- 最终审批: 通过审查后合并到主分支
处理审查反馈
🔧 常见问题解决
1. 合并冲突解决
2. 修改提交历史
3. CI 检查失败
4. 同步 Fork 仓库
📚 最佳实践
1. 保持 PR 简洁
- 一个 PR 只解决一个问题
- 避免包含不相关的变更
- 保持代码变更在合理范围内
2. 及时同步
- 定期同步上游仓库
- 及时解决合并冲突
- 保持分支整洁
3. 完善的测试
- 为新功能添加测试
- 确保测试覆盖率
- 包含边界情况测试
4. 清晰的文档
- 更新 API 文档
- 添加使用示例
- 更新 CHANGELOG
🎯 质量标准
代码质量
- 遵循 ESLint 规则
- 保持一致的代码风格
- 适当的注释和文档
- 合理的变量和函数命名
性能标准
- 避免不必要的重新渲染
- 优化大数据量处理
- 合理使用缓存机制
- 监控内存使用
兼容性
- 支持主流浏览器
- React 版本兼容性
- TypeScript 严格模式
- 无障碍访问支持
📞 获得帮助
如果在 PR 过程中遇到问题:
- 查看开发指南
- 搜索相关 Issues
- 在 Discussions 中提问
- 联系维护者
感谢您的贡献!每个 PR 都让 agentic-ui 变得更好。🎉
