【问题标题】:Do a code review on a whole project without a pull request在没有拉取请求的情况下对整个项目进行代码审查
【发布时间】:2020-02-05 13:01:03
【问题描述】:

我正在教编程,我想查看/评论我学生的代码。

最简单的方法是创建一个拉取请求,但是当学生是初学者时,他们对分支不熟悉,所以对他们来说并不那么容易。

已经有类似的问题,但他们没有答案,而且有点不同,因为前this one 是关于审查更改,而不是整个项目。

我希望有一个使用 Github 的解决方案,我认为目前这是不可能的as it is an open issue,但我对 hacky 解决方案或其他工具持开放态度。重要的是能够逐行注释代码。

我正在寻找的解决方案需要:

  • 允许在一个分支或主控的所有文件的每一行添加 cmets
  • 不需要拉取请求
  • 不需要比较分支
  • 不需要安装 Git 以外的其他软件

【问题讨论】:

  • 必须查看哪些工具?正如您现在所问的那样 - 您的问题与 SO 无关,您正在寻求“关于书籍、软件库或其他场外资源的建议”。
  • "不需要拉取请求" 是的,但是由于他们正在处理代码和学习编程,因此他们还学习基本的版本控制,创建分支也许是个好主意,并提交 PR(或在其他一些平台上的合并请求)。
  • 例如我听说过 Gerrit 或 Crucible,但我发现这些工具要么配置起来很耗时,要么需要安装软件,我不愿意这样做。我在这里问这个问题是因为我认为 Github 可能有一个我不知道的功能可以解决我的问题。
  • 无论如何,也许你需要 Atlassian 的 Crucible 之类的东西(请参阅 What's the difference between Crucible and Bitbucket Server?)。我记得你仍然需要设置一个存储库并将 Crucible 指向它,但它确实支持审查不在 PR 中的分支、文件、补丁和评论行。
  • @GinoMempin 凡事都有时间,对于某些人来说,一次学习所有这些东西是很困难的,而我作为老师的工作就是尽可能地让它变得容易理解。感谢Crucible的推荐,虽然我不太愿意安装软件。我编辑了我的问题以反映这一点!

标签: git review


【解决方案1】:

使用 Github 和您规定的单个分支,您可以为每个 cmets 创建问题,将它们链接到有问题的代码行或代码块。您可以通过突出显示代码并将更新的 url 复制到创建的问题或使用下拉列表到 Reference in new issue 来做到这一点。这使代码保持不变,并且都可以在 github Web 界面中完成/查看。

您可以创建不同的问题标签来指示评论/问题的严重性。

【讨论】:

  • 好主意!如果需要添加大量 cmets,这有点让人不知所措,因为发布/编辑问题的时间比 cmets 长。但它有效,谢谢!
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2022-07-11
  • 2019-06-02
  • 1970-01-01
  • 2010-10-03
  • 1970-01-01
  • 2016-06-11
相关资源
最近更新 更多