Page MenuHomePhabricator

arc alias outputs a confusing message when adding a shell command alias
Closed, ResolvedPublic

Description

arc alias outputs a confusing message when adding a shell command alias

WHAT HAPPENS
$ arc alias foo '!foo'
Aliased 'arc foo' to 'arc !foo'.

WHAT SHOULD HAPPEN
$ arc alias foo '!foo'
Aliased 'arc foo' to 'foo'.

Note that when running 'arc foo', it correctly runs 'foo'.

Revisions and Commits

Restricted Differential Revision

Event Timeline

Arcank updated the task description. (Show Details)
Arcank added a project: Arcanist.
Arcank added a subscriber: featherless.
Arcank added a subscriber: Arcank.
epriestley triaged this task as Wishlist priority.Nov 30 2015, 6:34 PM
epriestley added a subscriber: epriestley.

Aliased 'arc foo' to 'foo'.

I think we could be clearer than this. It could be misleading in cases like this:

Aliased 'arc review' to 'diff'.

Without context, I might expect arc review to now mean arc diff, when it really just means diff.

It would be more clear for the output to be explicit:

Aliased 'arc review' to shell command 'ls'.

Note that the comment when removing the alias has also this "issue":

$ arc alias foo
'arc foo' is currently aliased to 'arc !foo'.

Right, adding the "shell command" words would be nice.

epriestley added a revision: Restricted Differential Revision.Dec 3 2015, 2:04 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Dec 4 2015, 2:31 AM
epriestley added a commit: Restricted Diffusion Commit.