My comments below: On Mon, Oct 31, 2016 at 4:07 PM, Flavio Junqueira <f...@apache.org> wrote:
> > Feel free to contribute to my changes and suggest a different way in the > jira. We can definitely work together on this, I just want to have this > working soon. > Ok. I don't want to slow down your current work in progress, so I will try to see IF/WHAT I can contribute, but will let you know before hand. :) > > The script is bash, not python. Yup, in fact, I wrote the script with the intention of adding the option of attaching the diff to the JIRA issue to zk-merge-pr.py tool. It woud be for the sake of documeting the patch in the JIRA. Maybe I will create a new issue proposing this once we have the other things sorted out, and only if you guys think this is worth doing. > It doesn't make it patch available mainly because if we make it patch > available, then it will trigger the Jira QA, which will find no patch. It > is a bit messy to trigger this second build, so I'm reluctant in doing it. > I can see two options: > > - Only do github pull requests, in which case "Patch Available" in our > workflow means pull request available > - Somehow detect that there is a pull request and no trigger the Jira QA > IMHO, the the first option is less brittle and effective. Edward