Pull Request流程

Pull Request流程概述

Pull Request（简称PR）是Git协作开发中的核心机制，主要用于代码审查和功能合并。开发者通过PR向目标分支提交变更请求，团队成员可讨论修改后再决定是否合并。这种流程在GitHub、GitLab等平台已成为标准实践。

创建Pull Request前的准备

在发起PR前需要完成以下准备工作：

1. Fork仓库（适用于开源项目协作）：

   # 在GitHub界面点击Fork按钮
   git clone https://github.com/your-username/repository.git
   cd repository
   git remote add upstream https://github.com/original/repository.git
   

2. 创建特性分支：

   git checkout -b feature/new-login-form
   

3. 提交原子性变更：

   // 示例：登录表单组件修改
   // src/components/LoginForm.js
   const LoginForm = () => {
     const [email, setEmail] = useState('');
     const [password, setPassword] = useState('');
     
     return (
       <form>
         <input 
           type="email" 
           value={email}
           onChange={(e) => setEmail(e.target.value)}
           aria-label="Email address"
         />
         {/* 密码输入框... */}
       </form>
     );
   }
   

   提交时使用明确的消息：

   git commit -m "feat: add accessible email input to login form"
   

发起Pull Request

1. 推送分支到远程：

   git push origin feature/new-login-form
   

2. 在GitHub界面操作：

   • 进入仓库的"Pull requests"标签页
   • 点击"New pull request"按钮
   • 选择正确的base分支（通常是main/master）和compare分支

3. 填写PR模板：

   ## 变更描述
   - 新增无障碍邮箱输入框
   - 修复密码框的tab顺序问题
   
   ## 相关Issue
   Fixes #123
   
   ## 测试步骤
   1. 进入登录页面
   2. 验证邮箱输入框可被屏幕阅读器识别
   3. 使用Tab键测试焦点顺序
   

PR审查阶段

1. 自动检查流程：

   • CI/CD流水线运行测试
   • Codecov等工具检查测试覆盖率

   # 示例GitHub Actions配置
   name: CI
   on: [pull_request]
   jobs:
     test:
       runs-on: ubuntu-latest
       steps:
         - uses: actions/checkout@v2
         - run: npm install && npm test
   

2. 审查者操作：

   • 行内评论特定代码

   // 建议修改：增加输入长度限制
   <input 
     maxLength={320}  // 最大邮箱长度
     /* 其他属性 */
   />
   

   • 请求变更（Request changes）
   • 批准（Approve）

3. 作者处理反馈：

   • 在原有分支继续提交

   git add src/components/LoginForm.js
   git commit -m "fix: add maxLength to email input"
   git push origin feature/new-login-form
   

解决合并冲突

当目标分支有更新时可能出现冲突：

1. 同步上游变更：

   git fetch upstream main
   git merge upstream/main
   # 或使用rebase
   git rebase upstream/main
   

2. 手动解决冲突：

   <<<<<<< HEAD
   const MAX_EMAIL_LENGTH = 320;
   =======
   const MAX_EMAIL_LENGTH = 254;  // RFC标准
   >>>>>>> upstream/main
   

   修正后：

   const MAX_EMAIL_LENGTH = 254;  // 采用RFC标准
   

高级PR策略

1. Draft PR（草稿模式）：

   # GitHub CLI创建草稿PR
   gh pr create --draft --title "WIP: new login form" --body "Initial implementation"
   

2. 多提交整理：

   # 交互式rebase整理提交历史
   git rebase -i HEAD~3
   # 将多个提交squash为一个
   

3. 分支保护规则：

   • Require approvals（至少2个批准）
   • Require status checks（必须通过CI）
   • Require linear history（禁止merge commit）

PR合并后的操作

1. 清理分支：

   git checkout main
   git branch -d feature/new-login-form
   git push origin --delete feature/new-login-form
   

2. 同步fork仓库：

   git fetch upstream
   git merge upstream/main
   git push origin main
   

3. 版本标签：

   git tag -a v1.2.0 -m "Release login form updates"
   git push origin v1.2.0
   

企业级PR实践

1. 关联项目管理工具：

   ### JIRA关联
   [PROJ-123] Login form accessibility
   

2. 强制检查项：

   • SonarQube质量门禁
   • 安全扫描（如Snyk）

   # 安全扫描示例
   npm audit
   snyk test
   

3. 大型PR拆分策略：

   graph TD
     A[架构调整] --> B[API模块]
     A --> C[前端组件]
     B --> D[用户服务PR]
     C --> E[登录页PR]
   

移动端PR特殊考量

1. 构建产物检查：

   # React Native示例
   cd ios && pod install
   npx react-native run-ios --simulator="iPhone 13"
   

2. 多平台兼容性：

   // 平台特定代码标记
   if (Platform.OS === 'android') {
     // Android特有逻辑
   }
   

3. 应用尺寸监控：

   # 检查APK/IPA大小变化
   du -h android/app/build/outputs/apk/release/app-release.apk