[Rev 01] RFR: git-pr: support git pr create --draft
Robin Westberg
rwestberg at openjdk.java.net
Wed Feb 19 10:15:10 UTC 2020
On Wed, 19 Feb 2020 10:14:51 GMT, Erik Helin <ehelin at openjdk.org> wrote:
>> Hi all,
>>
>> please review this patch adds support for `--draft` switch to `git pr create`.
>>
>> Testing:
>> - Manual testing of `git pr create`
>>
>> Thanks,
>> Erik
>
> The pull request has been updated with 1 additional commit.
Looks good, just a naming comment, can keep as is as well! But remember to update TestPullRequest as well.
forge/src/main/java/org/openjdk/skara/forge/PullRequest.java line 138:
> 137: boolean isDraft();
> 138: void makeNotDraft();
> 139: }
While symmetrical, perhaps `makeFinal` or `makeReady` would look nicer.. But if so `isDraft` should perhaps be `isFinal` as well.
-------------
Marked as reviewed by rwestberg (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/454
More information about the skara-dev
mailing list