diff --git a/args/src/main/java/org/openjdk/skara/args/Argument.java b/args/src/main/java/org/openjdk/skara/args/Argument.java index 54511818e..7e4720821 100644 --- a/args/src/main/java/org/openjdk/skara/args/Argument.java +++ b/args/src/main/java/org/openjdk/skara/args/Argument.java @@ -65,7 +65,7 @@ public boolean asBoolean() { } public String asString() { - return via(Function.identity()); + return value == null ? null : via(Function.identity()); } public Argument or(int value) { diff --git a/args/src/main/java/org/openjdk/skara/args/ArgumentParser.java b/args/src/main/java/org/openjdk/skara/args/ArgumentParser.java index 1890cfa7d..703abb865 100644 --- a/args/src/main/java/org/openjdk/skara/args/ArgumentParser.java +++ b/args/src/main/java/org/openjdk/skara/args/ArgumentParser.java @@ -161,7 +161,7 @@ public Arguments parse(String[] args) { if (arg.contains("=")) { var parts = arg.split("="); var name = parts[0].substring(2); // remove leading '--' - var value = parts[1]; + var value = parts.length == 2 ? parts[1] : null; var flag = lookupFullname(name); values.add(new FlagValue(flag, value)); seen.add(flag); diff --git a/cli/src/main/java/org/openjdk/skara/cli/pr/GitPrSet.java b/cli/src/main/java/org/openjdk/skara/cli/pr/GitPrSet.java index 56dee69dc..c6c7ea7c9 100644 --- a/cli/src/main/java/org/openjdk/skara/cli/pr/GitPrSet.java +++ b/cli/src/main/java/org/openjdk/skara/cli/pr/GitPrSet.java @@ -115,7 +115,9 @@ public static void main(String[] args) throws IOException, InterruptedException var pr = getPullRequest(uri, repo, host, id); var assigneesOption = getOption("assignees", "set", arguments); - if (assigneesOption != null) { + if (assigneesOption == null) { + pr.setAssignees(List.of()); + } else { var usernames = Arrays.asList(assigneesOption.split(",")); var assignees = usernames.stream() .map(u -> host.user(u))