-
-
Notifications
You must be signed in to change notification settings - Fork 860
Document backport and cherry-pick labels #126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
+1 from me. We probably need to wait for a bit to settle on the exact workflow before starting to document it. We could probably cover the GitHub part in http://cpython-devguide.readthedocs.io/committing.html#backporting-changes-to-python-3-6-or-older-version (or split it to two separate subsections?)
+1. This is also my assumption. |
- reference the Git bootcamp page - recommend cherry_picker.py - explain backport/cherry-pick labels Closes #126
- reference the Git bootcamp page - recommend cherry_picker.py - explain backport/cherry-pick labels Closes #126
- reference the Git bootcamp page - recommend cherry_picker.py - explain backport/cherry-pick labels Closes python#126
- reference the Git bootcamp page - recommend cherry_picker.py - explain backport/cherry-pick labels Closes python/devguide#126
There should be a documentation explaining what to do with the labels
needs backport to X.Y
andcherry-pick for X.Y
. Perhaps as a new section in committing.rst?I think this is the current workflow: (as discussed in python/core-workflow#23)
needs backport to X.Y
label to the PR againstmaster
.master
PR is approved and merged, create the cherry-pick PR.needs backport to X.Y
label frommaster
PRcherry-pick for X.Y
to the cherry-pick PR.[X.Y]
Where
X.Y
is one of the maintenance branches, e.g.2.7
,3.5
,3.6
Related question: Can I assume that the core developer who merged the
master
PR will do the backport?Thanks :)
The text was updated successfully, but these errors were encountered: