關閉拉取請求的策略#

注意

這完全複製自舊的 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.