This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author cheryl.sabella
Recipients Mariatta, cheryl.sabella, docs@python, terry.reedy
Date 2017-05-02.21:36:31
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAACPsK5LD2Tz0E1U4XoDNGgsLuO551fukf=nMx+FDjY6wnZ2ZQ@mail.gmail.com>
In-reply-to <1493659505.42.0.433733083562.issue30208@psf.upfronthosting.co.za>
Content
Thank you for that information, Mariatta.  I had to read your description
twice to find the 9dc2b38 commit hash.  Once someone learns this, then they
will remember it, but you may want to highlight it on the cherry-pick
readme.  That was very well written and easy to follow, but I don't
remember it specifying the commit hash to use.

Cherry-pick is an awesome tool.  Can't wait to use it again!   :-)

On Mon, May 1, 2017 at 1:25 PM, Mariatta Wijaya <report@bugs.python.org>
wrote:

>
> Mariatta Wijaya added the comment:
>
> One PR per branch is correct.
>
> For future reference, preferably the commit hash to be cherry-picked is
> the one that is merged into CPython's master branch. It can be found at the
> original PR where it says <coredeveloper> merged commit <hash> into
> python:master. For PR 1353, it is 9dc2b38.
>
> For this issue, it doesn't make much difference since the PR contains only
> one commit. For PRs that contain multiple commits, it makes more sense to
> cherry-pick the squashed commit to master :)
>
> Thanks :)
>
> ----------
> nosy: +Mariatta
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <http://bugs.python.org/issue30208>
> _______________________________________
>
History
Date User Action Args
2017-05-02 21:36:31cheryl.sabellasetrecipients: + cheryl.sabella, terry.reedy, docs@python, Mariatta
2017-05-02 21:36:31cheryl.sabellalinkissue30208 messages
2017-05-02 21:36:31cheryl.sabellacreate