git - 基于主干开发的代码审查

标签 git pull-request

在进行基于主干的开发时,所有团队成员(据我所知)应该直接推送到 Master(或 main/trunk,无论你怎么调用它)。如图所示:

Trunk based development for a small team

Source

关于这种方法,我不明白代码审查如何适用。您当然可以在将任何 pull 请求 merge 到 Release 之前进行代码审查,但我认为这会相当大你必须做的评论。

是否可以暂停对 Master 的传入推送,直到完成代码审查?

或者在向 Master 发出 pull 请求时,是否有必要创建一个单独的分支(可能每个开发人员一个)来推送然后进行代码审查?

更新:

假设您对所有内容都使用功能分支,您当然可以只从功能分支执行 pull 请求并对其进行代码审查(如 this post 中所述)。但这要求一切都在功能分支中。据我了解,基于主干的开发的一部分是,并非所有内容都需要功能分支。

最佳答案

trunkbaseddevelopment.com 有一个关于代码审查的页面,它是这样说的:

In Trunk-Based Development teams, the PR should be on a short-lived feature branch and processed very quickly by reviews towards merging back to trunk/master.

https://trunkbaseddevelopment.com/continuous-review/

关于git - 基于主干开发的代码审查,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/62952712/

相关文章:

java - EGit - "Replace With Commit..."无法正常工作

git - 在 Git Bash for Windows 中, "standard input mode"对 `git commit -F -` 有什么用?

github - github上的"This comparison is taking too long to generate."错误

github - 有没有办法建议对 PR 进行修改?

git - 将 sublime text 3 设置为 git commit 文本编辑器

linux - git - "ignore"或避免版本控制子目录

git - 从 Git 中提取失败并给我以下错误 : client_global_hostkeys_private_confirm: server gave bad signature for RSA key 0

github - 要求在没有 "write"访问权限的分支保护规则中进行 PR 审查

git - 完全还原 pull 请求

azure-devops - 不要为 Azure DevOps 中已有拉取请求的分支触发构建