[
https://issues.apache.org/jira/browse/ZOOKEEPER-2597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15607121#comment-15607121
]
Benjamin Reed commented on ZOOKEEPER-2597:
------------------------------------------
i used the script to commit the pull request. i also followed your
instructions. here are a couple of things that went wrong:
1) you have the wrong repo line for adding the apache repo. it should be:
{code:borderStyle=solid}
git remote add apache https://git-wip-us.apache.org/repos/asf/zookeeper.git
{code}
2) when things go bad it doesn't delete the branches it creates. i'm not sure
if that is a bug or a feature. we should document that you need to remove the
temporary branches before rerunning the script.
3) the script asks {{List pull request commits in squashed commit message?
(y/n):}} i think the answer should be {{n}}
4) after the script ran i was very disappointed that the jira integration
didn't work. we should make sure we run the following before running the script:
{code}
sudo pip install jira
{code}
> Add script to merge PR from Apache git repo to Github
> -----------------------------------------------------
>
> Key: ZOOKEEPER-2597
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2597
> Project: ZooKeeper
> Issue Type: Improvement
> Reporter: Edward Ribeiro
> Assignee: Edward Ribeiro
> Priority: Minor
> Attachments: ZOOKEEPER-2597.patch
>
>
> A port of kafka-merge-pr.py to workon on ZooKeeper repo.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
