juju gui now prints the URL by default #6984

Merged
merged 2 commits into from Feb 15, 2017

Conversation

Projects
None yet
3 participants
Owner

wallyworld commented Feb 15, 2017

Description of change

Invert the behaviour of the juju gui command. The command will now just print the GUI URL by default. Using --browser is needed to launch the browser.

--show-credential (singular) is used instead of --show-credentials to be consistent with the rest of Juju

QA steps

bootstrap juju
juju gui --browser

Documentation changes

CLI doc for juju GUI will change

Bug reference

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

axw approved these changes Feb 15, 2017

cmd/juju/gui/gui_test.go
"github.com/juju/juju/cmd/juju/gui"
jujutesting "github.com/juju/juju/juju/testing"
coretesting "github.com/juju/juju/testing"
+ "github.com/juju/version"
@axw

axw Feb 15, 2017

Member

move me

Owner

wallyworld commented Feb 15, 2017

$$merge$$

Contributor

jujubot commented Feb 15, 2017

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

@jujubot jujubot merged commit a57fd1b into juju:2.1 Feb 15, 2017

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