[Rev 01] RFR: git-pr: support git pr create --draft
Erik Helin
ehelin at openjdk.java.net
Thu Feb 20 14:52:40 UTC 2020
On Wed, 19 Feb 2020 10:01:47 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
>> The pull request has been updated with 1 additional commit.
>
> 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.
Yeah, I sort of agree. However both platforms (GitHub and GitLab) does not have a word for the "not draft" state. So to make it very clear I think I prefer the current verbose (and slightly weird) naming to be make the code clearer.
-------------
PR: https://git.openjdk.java.net/skara/pull/454
More information about the skara-dev
mailing list