GitHub - LGTM并添加评论

Question

我收到PR并去测试，研究等。

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

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

所以我的关键字LGTM应该触发批准，但是其他评论会让人发现LGTM（所以在这种情况下不允许附加评论）？

此外，添加更多详细信息（例如上述内容）是一种好习惯吗？

Answer 1

  

所以我的关键字LGTM应该触发批准，但是其他评论会搞砸找到LGTM（所以在这种情况下不允许附加评论）？

不，根据official readme：

  

项目维护人员可以通过评论拉取请求和包括 LGTM（对我来说很好）在他们的批准文本中表明他们的批准。

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

  

此外，添加更多详细信息（例如上述内容）是一种好习惯吗？

如果更多细节会有所帮助，那么肯定。但没有理由只是为了它而添加它们。