1
villivateur 2021-12-03 08:42:56 +08:00 via Android
你可以拉取他的分支,然后在本地调试一下
|
2
SingeeKing 2021-12-03 08:46:07 +08:00 via iPhone
看代码:clone 分支看
查看是否按期望运行:CI |
3
Juszoe 2021-12-03 09:20:23 +08:00
别人提交 PR 到你的项目的话,github 会帮你创建分支的,直接拉取之后审核就行
|
4
shanyuhai123 2021-12-03 09:24:32 +08:00
1. 代码分批提交,一次合太多无关联的打回
2. 描述内容清晰且相对完整(项目不熟悉的人也能够快速了解这次解决了什么,发生了什么变化,可关联 issue ) 3. 你们需要补充测试(这个挺难) |
5
wszgrcy OP |
6
crayygy 2021-12-03 09:31:16 +08:00
1. 人工 review ,多一双眼睛多一份安心
2. UT ,多一个 UT 多一份保障 3. 自动化 CI 合并 PR 之前跑一遍编译和测试 上面三个都能做到的话,bug 肯定还会有,但是一些明显的错误还是能快速发现的。 |
7
wszgrcy OP 找到了,git fetch origin pull/编号 /head 可以拉取下来指定的 pr
|
8
2i2Re2PLMaDnghL 2021-12-03 09:39:34 +08:00
> 如果改的比较多,并且不知道期望运行是否正确,应该如何实现呢?
这个就是 CI 的活,PR 触发 CI test 成本上考虑可以先为典型路线进行 test —— 本地处理,可以 clone 他的 repo (更精细的是把他的 repo 注册为一个 downstream remote 再 fetch ),然后用 IDE 看 commit 和 diff |
9
Vegetable 2021-12-03 09:46:24 +08:00
git checkout -b 新分支 基础分支
git pull https://github.com/对方 ID/仓库 发起 PR 的分支 |
10
lance6716 2021-12-03 09:53:46 +08:00 via iPhone
装一下 GitHub cli ,然后 gh pr 1234 就切换好了
|
11
otakustay 2021-12-03 11:24:22 +08:00
他的 PR 也是一个分支,你还能发一个 PR 到他的分支上做 PR 的 PR
|
12
SoloCompany 2021-12-03 13:24:57 +08:00 via iPhone 1
@otakustay 不需要的,除非额外设置,fork 分支,在创建 pr 的时候默认会给目标拥有者 push 权限
|