RFR: Add support for adding an issue when formatting the commit message
Erik Helin
ehelin at openjdk.org
Fri Sep 20 13:14:00 UTC 2019
On Thu, 19 Sep 2019 14:59:52 GMT, Erik Helin <ehelin at openjdk.org> wrote:
> On Thu, 19 Sep 2019 12:11:49 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
>
>> Hi all,
>>
>> Please review this change that allows setting the issue of a commit through the PR title.
>>
>> Best regards,
>> Robin
>>
>> ----------------
>>
>> Commits:
>> - 6222eb97: Add support for adding an issue when formatting the commit message
>>
>> Stats: 97 lines in 8 files changed: 78 ins; 4 del; 15 mod
>> Changes: https://git.openjdk.java.net/skara/pull/148/files
>> Webrev: https://webrevs.openjdk.java.net/skara/148/webrev.00
>> Patch: https://git.openjdk.java.net/skara/pull/148.diff
>> Fetch: git fetch https://git.openjdk.java.net/skara pull/148/head:pull/148
>
> host/src/main/java/org/openjdk/skara/host/github/GitHubPullRequest.java line 217:
>
>> 216: public void setTitle(String title) {
>> 217: throw new RuntimeException("not implemented yet");
>> 218: }
>
> Won't this be an issue if we run the unit tests as integration tests against real hosts?
>
> host/src/main/java/org/openjdk/skara/host/gitlab/GitLabMergeRequest.java line 259:
>
>> 258: public void setTitle(String title) {
>> 259: throw new RuntimeException("not implemented yet");
>> 260: }
>
> Same issue here?
Ah ok, lets tackle it later then ð
PR: https://git.openjdk.java.net/skara/pull/148
More information about the skara-dev
mailing list