cmd/juju: Use the new visitwebpage function to allow usso login via the cli #4623

Merged
merged 3 commits into from Mar 14, 2016

Conversation

Projects
None yet
4 participants
Member

mattyw commented Mar 4, 2016

jujuclient/oauth.go
+
+// StoreOAuthToken is the location where the charm store oauth token is
+// expected to be found.
+func StoreOAuthToken() string {
@alesstimec

alesstimec Mar 4, 2016

Member

nitpick: StoreOAuthTokenLocation

Member

alesstimec commented Mar 4, 2016

LGTM! well done!

Member

mattyw commented Mar 14, 2016

$$merge$$

Contributor

jujubot commented Mar 14, 2016

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

Member

mattyw commented Mar 14, 2016

$$merge$$

Contributor

bz2 commented Mar 14, 2016

Build failed: Matty keeps asking for it to be cancelled.

Contributor

bz2 commented Mar 14, 2016

$$merge$$

Contributor

jujubot commented Mar 14, 2016

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

Contributor

jujubot commented Mar 14, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/6819

Member

mattyw commented Mar 14, 2016

$$merge$$

Contributor

jujubot commented Mar 14, 2016

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

jujubot added a commit that referenced this pull request Mar 14, 2016

Merge pull request #4623 from mattyw/18-non-interactive-login
cmd/juju: Use the new visitwebpage function to allow usso login via the cli



(Review request: http://reviews.vapour.ws/r/4063/)

@jujubot jujubot merged commit fa7a9e2 into juju:master Mar 14, 2016

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