Rename 'attach' to 'attach-resource'. #7693

Merged
merged 2 commits into from Aug 1, 2017

Conversation

Projects
None yet
3 participants
Member

anastasiamac commented Aug 1, 2017

Description of change

This command was overlooked in major rename effort that occurred on a cusp of Juju 2.x.
This is needed to comply with standard naming convention and to add clarity to command intent.

Previous command name 'attach' is now an alias for 'attach-resource' for backward compatibility.

QA steps

  1. 'juju help commands' has both 'attach' and 'attach-resource'
  2. running 'juju attach-resource' yields the same result as running 'juju attach'

Documentation changes

Yes!
All references to 'juju attach' in documentation should be changed to 'juju attach-resource'.

Bug reference

Part of https://bugs.launchpad.net/juju/+bug/1707564

featuretests/cmd_juju_resources_test.go
@@ -92,7 +92,7 @@ Resource Revision
)
// check "juju attach"
@hmlanigan

hmlanigan Aug 1, 2017

Member

nit: update the comment?

Member

anastasiamac commented Aug 1, 2017

$$merge$$

Contributor

jujubot commented Aug 1, 2017

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

@jujubot jujubot merged commit 239712d into juju:develop Aug 1, 2017

1 check passed

github-check-merge-juju Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details

@anastasiamac anastasiamac deleted the anastasiamac:rename-attach-resource branch Aug 1, 2017

Member

anastasiamac commented Aug 23, 2017

@pmatulis,

This change may require documentation update.

@pmatulis pmatulis referenced this pull request in juju/docs Sep 15, 2017

Closed

[2.3] review Resources concept #2066

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