1809214 - `moz-phab uplift --train` should take a shortname instead of callsign

Status: RESOLVED FIXED

(Conduit :: moz-phab, enhancement)

Description

[Connor Sheehan [:sheehan]]

We currently use the callsign to indicate which repo is used for uplift. We should make moz-phab take the repo shortname instead as it is more readable due to callsigns being limited to characters, therefore esr91 is instead ESRNINEONE. It would also align with what we use in the Lando web UI.

Comment 1

[Connor Sheehan [:sheehan]]

Attachment: uplift: use repo shortnames instead of callsigns for `--train` (Bug 1809214) r?grammar,zeid

The current implementation of the train parameter in moz-phab uses
the repo callsign to specify the uplift target. The shortname is
what the web UI uses and is the nicer name value we usually refer
to repos with.

Rename get_repository to get_repository_by_callsign to clarify
the input parameter and add a get_repository_by_shortname function.
Update list_trains to print the shortnames of the uplift repos instead
of the callsign. Then update map_train_arg_to_repo to use the
shortname to find the repo information on Phabricator and have the
callsign set from the returned information instead of from
args.train directly.