求助 开源贡献的流程
github吧
全部回复
仅看楼主
level 3
各位好,我提交了一个pr到cpython上,然后review之后修改代码是不是不好用rebase+push f?会影响review。那么用merge吗
2025年04月30日 03点04分 1
吧务
level 12
引用 @贴吧用户_GWMRtPE 在楼中楼的发言:
回复 Dragon1573 :133067 没什么意思就是一个文档的
——————————
看了一眼,你的PR只是修改了一个 reStructuredText 文档而已。
其他的 Contributors 应该和你提及过不要用 Force Push 了。
而且,你其实没有必要把 main 合并到你 PR 里面,CPython 仓库统一使用 Squash Merge ,它们会把你 PR 相对 main 的更改压缩到一个 Commit 中,即使 main 上有更多的更新也完全不影响。我在 microsoft/winget-pkgs 上做 PR ,那个更新频率每过大概1小时就在 master 上多出上百 Commit ,现在 winget-pkgs 的 PR 号已经超过25万了。
2025年04月30日 10点04分 0
截止当前,winget-pkgs 的 master 分支有 218,570 Commits ,比 CPython 的 126,708 Commits 多得多了[滑稽]
2025年04月30日 10点04分
level 3
还是说不用管main上的新commit,直接push就好了
2025年04月30日 03点04分 2
@Dragon1573 133067 没什么意思[小乖]就是一个文档的
2025年04月30日 08点04分
直接 Push 进去开 PR 即可,GitHub 计算 Branch Diff 是会把 main 上面那些新 Commit 给无视掉的
2025年04月30日 07点04分
你要 Rebase + Force Push ,也应该避免在 Force Push 之后改动到和你本次 PR 无关的东西。我在 microsoft/winget-pkgs 贡献榜排行历史第20,我 PR 都是直接开的,不一定是从 master 上的最新 Commit 分支出来的
2025年04月30日 07点04分
@Dragon1573 好的 谢谢你的解释[真棒]
2025年04月30日 08点04分
1