[
https://issues.apache.org/jira/browse/BOOKKEEPER-920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15259672#comment-15259672
]
ASF GitHub Bot commented on BOOKKEEPER-920:
-------------------------------------------
Github user sijie commented on the pull request:
https://github.com/apache/bookkeeper/pull/36#issuecomment-214989592
the change looks good to me +1
> Extend bk-merge-pr.py to add more info to Jira ticket when merging
> ------------------------------------------------------------------
>
> Key: BOOKKEEPER-920
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-920
> Project: Bookkeeper
> Issue Type: Improvement
> Reporter: Matteo Merli
> Assignee: Matteo Merli
> Priority: Minor
> Fix For: 4.4.0
>
>
> We should add more information to the Jira ticket when merging a github PR.
> Currently, the update on the Jira contains only the author and commit title.
> It would be good to also have:
> * Author
> * Committer
> * Reviewers (Better if we automatically read the '+1' from the PR comments)
>
> In addition, we should add the option to run the tests locally on the merged
> PR before pushing to apache git.
> Recently we had cases of PRs which were fine on their own, but then tests
> started failing after merging into master (because of other commits since),
> even without git merge conflicts.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
