RFR: Fix parsing of editor commandline in git-pr

[Christoph Langer]

On Wed, 20 Nov 2019 09:10:14 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> On Tue, 19 Nov 2019 14:35:13 GMT, Christoph Langer <clanger at openjdk.org> wrote:
> 
>> Fix parsing of editor commandline in git-pr.
>> 
>> ----------------
>> 
>> Commits:
>>  - 199285fc: Fix parsing of editor commandline in git-pr.
>> 
>> Changes: https://git.openjdk.java.net/skara/pull/259/files
>>  Webrev: https://webrevs.openjdk.java.net/skara/259/webrev.00
>>   Stats: 10 lines in 2 files changed: 8 ins; 0 del; 2 mod
>>   Patch: https://git.openjdk.java.net/skara/pull/259.diff
>>   Fetch: git fetch https://git.openjdk.java.net/skara pull/259/head:pull/259
> 
> Hi Christoph,
> 
> thanks for contributing! Just two small nits before I will continue to review this:
> 
> 1. Please describe what you changed in this pull request. You did not provide a description in this PR.
> 2. One usually do not end PR titles with a `.`, so please remove that from the title of the PR.
> 
> Thanks!
> Erik

> Hi Christoph,
> 
> thanks for contributing! Just two small nits before I will continue to review this:
> 
> 1. Please describe what you changed in this pull request. You did not provide a description in this PR.
> 2. One usually do not end PR titles with a `.`, so please remove that from the title of the PR.
> 
> Thanks!
> Erik

Hi Erik,

thanks for pointing out these points. I think I've addressed them.

I created the pull request via the Skara tool call "git pr create". I'm wondering whether there is any documentation about this tool?
E.g. is there an option to give a pull request description? Also, how is one supposed to refer to a JBS bug (if I created one).
Furthermore, git pr has quite a few suboptions. Is there some documentation on what these all are supposed to do?

Thanks
Christoph

PR: https://git.openjdk.java.net/skara/pull/259