如果你为GitHub仓库配置了DeepSource,在每个拉动请求中,都会对更改集的新增和修改文件进行分析。例如,在这个拉动请求中,问题是针对masonite/request.py 和masonite/response.py ,而不仅仅是针对新引入的行。看一下这里的运行情况。
我们采用了遵守童子军规则的方法:
童子军规则,离开营地时一定要比你发现时更干净。
这里的营地,曾经是所有被触及的文件--那样的话,鼓励开发者修复文件中的所有问题,即使只做了一行改动;最终导致一个更健康的代码基础。我们询问了我们的用户,他们不这么认为。原因是,拉动请求中的修改开始变得脱离了背景。拉动请求(或)变更列表的最佳做法是,将变更完全保持在功能/修复的上下文中,而不是超出它。
我们已经在最近的版本中推出了这项改进。从现在开始,在拉动请求中,将只报告新增加(或)修改的行和由于改变的行而在文件中新引入的问题。童子军规则仍然适用,但游乐场的范围是行,而不是文件。
GitHub / 拉动请求的变化
例如,在上图所示的拉动请求中,只有65 行被更改。因此,DeepSource检查将只报告以下问题。
- 行上的问题
65 - 由行中的更改引起的文件上的任何新引入的问题
65
DeepSource / History / Run
压制符合上下文的问题。干杯!