[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