关于关闭拉取请求的政策#

注意

这是从旧 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.