Integrated: Do not print "Optional[repo name]" for backport PR command

Mon Oct 11 17:51:30 UTC 2021

On Thu, 7 Oct 2021 12:48:43 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> The sneaky `var` notation has caused a type problem. :-( `var repoName` was in fact `Optional<String>`, not `String`, so when printing this to the user, it was presented like this:
> 
> "Could not automatically backport 481a0456 to Optional[openjdk/jdk11u-dev] due to conflicts in the following files:"
> 
> The naming here is perhaps not my proudest moment, but I did not want to mess with the usage by `potentialTargetRepo` and so I wanted to keep the original value. I think the else-string will never actually show up, due to the isEmpty() check on potentialTargetRepo.

This pull request has now been integrated.

Changeset: 42e04b17
Author:    Magnus Ihse Bursie <ihse at openjdk.org>
URL:       https://git.openjdk.java.net/skara/commit/42e04b179457d4201ad640a55b7fea3c275a7b46
Stats:     3 lines in 1 file changed: 1 ins; 0 del; 2 mod

Do not print "Optional[repo name]" for backport PR command

Reviewed-by: erikj

-------------

PR: https://git.openjdk.java.net/skara/pull/1232