关闭拉取请求的策略

注意

这完全复制自旧的 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.