github - GitHub - LGTM 和添加评论

Question

我收到了 PR，然后去测试、研究等。

它看起来不错，但我想在WRITE框中添加详细信息（通常我们只放置 LGTM）：

1）“我像这样测试了函数 X 和 Y……”
2）“这里是 Y 在线文档的链接，描述了为什么这是一个好主意……”
3）“LGTM”

所以我的关键字LGTM应该触发批准，但额外的评论是否会混淆查找 LGTM（所以在这种情况下，不允许额外的评论）？

此外，添加更多细节（例如如上所述）是否是一种好习惯？

标签： github

Answer 1

因此，我的关键字 LGTM 应该会触发批准，但其他评论是否会混淆查找 LGTM（因此在这种情况下，不允许添加评论）？

不，根据官方自述文件：

项目维护者可以通过评论拉取请求并在他们的批准文本中包含LGTM（对我来说看起来不错）来表明他们的批准。

强调我的 - 请注意 LGTM 只需要包含，它不必是唯一的评论。

此外，添加更多细节（例如如上所述）是否是一种好习惯？

如果更多细节会有所帮助，那么当然可以。但是没有理由仅仅为了它而添加它们。