Accepting Pull Requests#
This page is a step-by-step guide for core developers who need to assess, merge, and possibly backport a pull request on the main repository.
Assessing a pull request#
Before you can accept a pull request, you need to make sure that it is ready to enter the public source tree. Ask yourself the following questions:
- Are there ongoing discussions at the issue tracker?
Read the linked issue. If there are ongoing discussions, then we need to have a resolution there before we can merge the pull request.
- Was the pull request first made against the appropriate branch?
The only branch that receives new features is
main, the in-development branch. Pull requests should only target bug-fix branches if an issue appears in only that version and possibly older versions.
- Are the changes acceptable?
If you want to share your work-in-progress code on a feature or bugfix, then you can open a
WIP-prefixed pull request, publish patches on the issue tracker, or create a public fork of the repository.
- Do the checks on the pull request show that the test suite passes?
Make sure that all of the status checks are passing.
- Does the patch break backwards-compatibility without a strong reason?
Run the entire test suite to make sure that everything still passes. If there is a change to the semantics, then there needs to be a strong reason, because it will cause some peoples’ code to break. If you are unsure if the breakage is worth it, then ask on the Core Development Discourse category.
- Does documentation need to be updated?
If the pull request introduces backwards-incompatible changes (e.g. deprecating or removing a feature), then make sure that those changes are reflected in the documentation before you merge the pull request.
- Were appropriate labels added to signify necessary backporting of the pull request?
If it is determined that a pull request needs to be backported into one or more of the maintenance branches, then a core developer can apply the label
needs backport to X.Yto the pull request. Once the backport pull request has been created, remove the
needs backport to X.Ylabel from the original pull request. (Only core developers and members of the Python Triage Team can apply labels to GitHub pull requests).
- Does the pull request pass a check indicating that the submitter has signed the CLA?
Make sure that the contributor has signed a Contributor Licensing Agreement (CLA), unless their change has no possible intellectual property associated with it (e.g. fixing a spelling mistake in documentation). The CPython CLA Bot checks whether the author has signed the CLA, and replies in the PR if they haven’t. For further questions about the CLA process, write to firstname.lastname@example.org.
What's New in Pythonand
If the change is particularly interesting for end users (e.g. new features, significant improvements, or backwards-incompatible changes), then an entry in the
What's New in Pythondocument (in
Doc/whatsnew/) should be added as well. Changes that affect only documentation generally do not require a
NEWSentry. (See the following section for more information.)
Updating NEWS and What’s New in Python#
Almost all changes made to the code base deserve an entry in
If the change is particularly interesting for end users (e.g. new features,
significant improvements, or backwards-incompatible changes), then an entry in
What's New in Python document (in
Doc/whatsnew/) should be added
as well. Changes that affect documentation only generally do not require
There are two notable exceptions to this general principle, and they both relate to changes that:
Already have a
Have not yet been included in any formal release (including alpha and beta releases)
These are the two exceptions:
If a change is reverted prior to release, then the corresponding entry is simply removed. Otherwise, a new entry must be added noting that the change has been reverted (e.g. when a feature is released in an alpha and then cut prior to the first beta).
If a change is a fix (or other adjustment) to an earlier unreleased change and the original
NEWSentry remains valid, then no additional entry is needed.
If a change needs an entry in
What's New in Python, then it is very
likely not suitable for including in a maintenance release.
If you are unable to use the tool, then you can create the
NEWS entry file
Misc/NEWS.d directory contains a sub-directory named
next, which contains various sub-directories representing classifications
for what was affected (e.g.
Misc/NEWS.d/next/Library for changes relating
to the standard library). The file name itself should be in the format
<datetime>is today’s date joined with a hyphen (
-) to the current time, in the
<issue-number>is the issue number the change is for (e.g.
<nonce>is a unique string to guarantee that the file name is unique across branches (e.g.
Yl4gI2). It is typically six characters long, but it can be any length of letters and numbers. Its uniqueness can be satisfied by typing random characters on your keyboard.
As a result, a file name can look something like
The contents of a
NEWS file should be valid reStructuredText. An 80 character
column width should be used. There is no indentation or leading marker in the
-). There is also no need to start the entry with the issue
number since it is part of the file name. You can use
inline markups too. Here is an example of a
Fix warning message when :func:`os.chdir` fails inside :func:`test.support.temp_cwd`. Patch by Chris Jerdonek.
The inline Sphinx roles like
:func: can be used help readers
find more information. You can build HTML and verify that the
link target is appropriate by using make html.
While Sphinx roles can be beneficial to readers, they are not required.
``code blocks`` can be used instead.
As a core developer, you have the ability to push changes to the official Python repositories, so you need to be careful with your workflow:
You should not push new branches to the main repository. You can still use them in the fork that you use for the development of patches. You can also push these branches to a separate public repository for maintenance work before it is integrated into the main repository.
You should not commit directly into the
mainbranch, or any of the maintenance branches. You should commit against your own feature branch, and then create a pull request.
For a small change, you can make a quick edit through the GitHub web UI. If you choose to use the web UI, be aware that GitHub will create a new branch in the main CPython repository rather than in your fork. Delete this newly created branch after it has been merged into the
mainbranch or any of the maintenance branches. To keep the CPython repository tidy, remove the new branch within a few days.
Keep a fork of the main repository, since it will allow you to revert all local changes (even committed ones) if you’re not happy with your local clone.
Seeing active branches#
If you use
git branch, then you will see a list of branches. The only branch that receives new features is
main, the in-development branch. The other branches receive only
bug fixes or security fixes.
Backporting changes to an older version#
If it is determined that a pull request needs to be backported into one or
more of the maintenance branches, then a core developer can apply the label
needs backport to X.Y to the pull request.
After the pull request has been merged, miss-islington (bot) will first try to do the backport automatically. If miss-islington is unable to do it, then the pull request author or the core developer who merged it should look into backporting it themselves, using the backport generated by cherry_picker.py as a starting point.
You can get the commit hash from the original pull request, or you can use
git log on the
main branch. To display the 10 most recent commit
hashes and their first line of the commit, use the following command:
git log -10 --oneline
You can prefix the backport pull request with the branch, and reference
the pull request number from
main. Here is an example:
[3.9] gh-12345: Fix the Spam Module (GH-NNNN)
Here “gh-12345” is the GitHub issue number, and “GH-NNNN” is the number of the original pull request. Note that cherry_picker.py adds the branch prefix automatically.
Once the backport pull request has been created, remove the
needs backport to X.Y label from the original pull request. (Only
core developers and members of the Python Triage Team
can apply labels to GitHub pull requests).
Reverting a merged pull request#
To revert a merged pull request, press the
Revert button at the
bottom of the pull request. That will bring up the page to create a
new pull request where the commit can be reverted. It will also create
a new branch on the main CPython repository. Delete the branch once
the pull request has been merged.
Always include the reason for reverting the commit to help others understand why it was done. The reason should be included as part of the commit message. Here is an example:
Revert gh-NNNN: Fix Spam Module (GH-111) Reverts python/cpython#111. Reason: This commit broke the buildbot.