Warning message or no, I keep using 'eval' instead of 'php-eval', and therefore would like to make it a first-class alias again. Any objections?

I've pretty much given up on the others. Does anyone think we should remove any of the aliases we deprecated in drush-3?

CommentFileSizeAuthor
#3 drush-command-aliases.patch2.27 KBgreg.1.anderson
drush-eval-alias.patch688 bytesgreg.1.anderson

Comments

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented

I don't mind if we un-deprecate 'eval', and other substrings command names.

It could be also interesting to remove deprecated alias not related to current command names. Just those two:

pm-list:

    'aliases' => array('sm'),
    'deprecated-aliases' => array('statusmodules'),

pm-releases:

    'deprecated-aliases' => array('info'),
moshe weitzman’s picture

moshe weitzman commented

+1 for eval change and those in #1

greg.1.anderson’s picture

greg.1.anderson commented
StatusFileSize
new drush-command-aliases.patch2.27 KB

Okay, per #1 and #2, I have removed 'info' and 'statusmodules' from the code, and I have made 'sm' a deprecated alias (since we have allowed it up until now).

I put 'eval' back as an alias, and added 'ev' as a second shorter alias for the same function.

I added pml and pmi for pm-list and pm-info, and rel for pm-releases.

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented

perhaps rl instead of rel to align with rln. Anyway I've no hard objection to all of this.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs review » Fixed

Committed with 'rl' in place of 'rel'.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.