[Rev 01] RFR: 199: Set fixVersion in an issue when a commit is integrated
Erik Helin
ehelin at openjdk.java.net
Fri Dec 13 11:04:54 UTC 2019
On Fri, 13 Dec 2019 11:04:53 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
>> Hi all,
>>
>> Please review this change that sets fixVersion in a commit's issue after integration, if a version has been configured in the repository's .jcheck/conf file.
>>
>> Also sets a version for Skara itself!
>>
>> Best regards,
>> Robin
>
> The pull request has been updated with 1 additional commit.
bots/notify/src/main/java/org/openjdk/skara/bots/notify/IssueUpdater.java line 81:
> 80:
> 81: if (fixVersion != null) {
> 82: if (fixVersion.equals("<repo>")) {
How about turning this around a bit into:
if (fixVersion == null) {
// Try to find fixVersion from .jceck/conf
}
if (fixVersion != null) {
issue.get().addFixVersion(fixVersion)
}
If you don't sent the `fixVersion` via the bot configuration, then it will be `null`, and then we will read it from `.jcheck/conf`. This avoids the need for the special `<repo>` version.
-------------
PR: https://git.openjdk.java.net/skara/pull/324
More information about the skara-dev
mailing list