Added model-default as alias to model-defaults. #7726

Merged
merged 1 commit into from Aug 10, 2017

Conversation

Projects
None yet
4 participants
Member

anastasiamac commented Aug 10, 2017

Description of change

For usability, this PR adds an alias to 'model-defaults' command - 'model-default', singular.

QA steps

  1. 'juju help commands' lists 'model-default' as well.
  2. running 'juju model-default <model-config-param=param-value>' has the same effect as running 'juju model-defaults <model-config-param=param-value>'.

Documentation changes

A command alias has been added.

Bug reference

https://bugs.launchpad.net/juju/+bug/1656216

Member

anastasiamac commented Aug 10, 2017

$$merge$$

Contributor

jujubot commented Aug 10, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 2d30167 into juju:develop Aug 10, 2017

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details

@anastasiamac anastasiamac deleted the anastasiamac:model-defaults-alias branch Aug 10, 2017

Member

anastasiamac commented Aug 23, 2017

@pmatulis,

This change may require documentation update.

pmatulis commented Sep 6, 2017

@anastasiamac The commands.md page needs updating but that's a different matter. Other than that, the docs don't make any mention of command aliases (with the exception of a single page, actions.md). I think this is a wise approach (prevents clutter and cuts down on confusion).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment