关闭拉取请求的政策#
注意
这是从旧的 IPython wiki 直接复制过来的,目前正在开发中。开发指南的这一部分中的许多信息已经过时。
IPython 有关关闭拉取请求的政策如下。此政策的目标是保持我们的拉取请求队列较小,并使我们能够专注于正在积极开发且有较大机会很快合并到主分支的代码。
当以下情况发生时,拉取请求将被关闭:
已经过审核,但已等待提交者提交更多代码以解决评论问题一个月或更长时间。
评审过程发现了一些更大的设计或技术问题,这些问题超出了特定拉取请求的细节范围。
特别是,我们不接受不针对任何特定错误的整体大型“清理”更改。这包括尾随空白、PEP8等。原因之一是这种大规模清理提供了引入新且微妙的错误的大量机会。
通常情况下,我们不会因为缺乏审查而关闭拉取请求。如果一个拉取请求已经闲置了一个月或更长时间没有审查,我们需要提醒自己并开始审查它。
当一个拉取请求被关闭时,我们将执行以下操作:
在拉取请求中发布一条GitHub消息,以确认每个人都同意关闭拉取请求。此消息应引用此政策。
打开一个问题以跟踪拉取请求。此问题应描述重新打开拉取请求所需的内容。
在拉取请求中发布一条GitHub消息,鼓励提交者继续工作,并详细说明需要解决哪些问题以便重新打开拉取请求。
此政策在以下主题中进行了讨论:
https://mail.scipy.org/pipermail/ipython-dev/2012-August/010025.html
示例消息:#
Hi,
This PR has been inactive for 1 month now, so we are going to close it and open an
issue to reference it. We try to keep our pull request queue small and focused on
active work. We encourage you to reopen the pull request if and when you
continue to work on this. Please contact us if you have any questions.
Thanks for contributing.
see https://github.com/ipython/ipython/wiki/Dev%3A-Closing-pull-requests/ for
our policies on closing pull requests.