[
https://issues.apache.org/jira/browse/BOOKKEEPER-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Enrico Olivelli reopened BOOKKEEPER-1021:
-----------------------------------------
I am reopening the issue because I got this error while trying to merge a PR.
{code}
$$ python dev/bk-merge-pr.py
File "dev/bk-merge-pr.py", line 415
useremail = user['email'].strip() if user['email'] or None
^
SyntaxError: invalid syntax
{code}
I'm running Python 2.7.13
In order to run the tool I have changed the file locally but I'm not an expert
of Python so I will not submit a patch
> Improve the merge script to handle github reviews api
> -----------------------------------------------------
>
> Key: BOOKKEEPER-1021
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1021
> Project: Bookkeeper
> Issue Type: Bug
> Components: build
> Reporter: Sijie Guo
> Assignee: Sijie Guo
> Fix For: 4.5.0
>
>
> Current script doesn't handle the new reviews API.
> We need to improve the merge script to handle the new review API.
> https://developer.github.com/v3/pulls/reviews/
> We can do similar thing like what DistributedLog is doing.
> https://github.com/apache/incubator-distributedlog/blob/master/scripts/dev/dl-merge-pr.py
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
