Clearer this way. Very little code refers to this context by name.

CommentFileSizeAuthor
#3 drush-rename-options-context-to-cli-2.patch6.31 KBgreg.1.anderson
drush-rename-options-context-to-cli.patch5.17 KBgreg.1.anderson

Comments

greg.1.anderson’s picture

greg.1.anderson commented
Status: Active » Needs review
greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs review » Needs work
greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs work » Needs review
StatusFileSize
new drush-rename-options-context-to-cli-2.patch6.31 KB

The patch in #0 missed one of the 'options' references in the processing of command-specific options. Rather than change it to 'cli', I re-implemented the handling of --no options for better stability and compatibility with standard drush option precedence.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

Looks good. Thanks.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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