- 比如别人合到我项目上的 pr,如果是改的少,在 github 上就可以直接看就行了
- 如果改的比较多,并且不知道期望运行是否正确,应该如何实现呢?
- 个人想到的可能是直接去他的代码里 clone.....但是感觉有点暴力,并且如何环境复杂的话审核 pr 更麻烦了,不能每一个都这么做
- 问下经常评审代码的大佬,这个正规流程是啥?
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
This topic created in 1797 days ago, the information mentioned may be changed or developed.
 |
DawnMoonDec 3, 2021 via Android
1
你可以拉取他的分支，然后在本地调试一下
|
 |
SingeeKingDec 3, 2021 via iPhone
2
看代码：clone 分支看
查看是否按期望运行：CI |
 |
JuszoeDec 3, 2021
3
别人提交 PR 到你的项目的话，github 会帮你创建分支的，直接拉取之后审核就行
|
 |
shanyuhai123Dec 3, 2021
4
1. 代码分批提交，一次合太多无关联的打回
2. 描述内容清晰且相对完整（项目不熟悉的人也能够快速了解这次解决了什么，发生了什么变化，可关联 issue ） 3. 你们需要补充测试（这个挺难） |
 |
5
|
 |
crayygyDec 3, 2021
6
1. 人工 review ，多一双眼睛多一份安心
2. UT ，多一个 UT 多一份保障 3. 自动化 CI 合并 PR 之前跑一遍编译和测试 上面三个都能做到的话，bug 肯定还会有，但是一些明显的错误还是能快速发现的。 |
|
7
找到了,git fetch origin pull/编号 /head 可以拉取下来指定的 pr
|
 |
2i2Re2PLMaDnghLDec 3, 2021
8
> 如果改的比较多,并且不知道期望运行是否正确,应该如何实现呢?
这个就是 CI 的活，PR 触发 CI test 成本上考虑可以先为典型路线进行 test —— 本地处理，可以 clone 他的 repo （更精细的是把他的 repo 注册为一个 downstream remote 再 fetch ），然后用 IDE 看 commit 和 diff |
 |
VegetableDec 3, 2021
9
git checkout -b 新分支 基础分支
git pull https://github.com/对方 ID/仓库 发起 PR 的分支 |
 |
lance6716Dec 3, 2021 via iPhone
10
装一下 GitHub cli ，然后 gh pr 1234 就切换好了
|
 |
otakustayDec 3, 2021
11
他的 PR 也是一个分支，你还能发一个 PR 到他的分支上做 PR 的 PR
|
 |
12
@otakustay 不需要的，除非额外设置，fork 分支，在创建 pr 的时候默认会给目标拥有者 push 权限
|
1
Select Language