/cmd

Switch to using gopkg.in/yaml.v2 #22

Merged
merged 1 commit into from Oct 11, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@bz2 @wallyworld @jujubot
@bz2
Contributor

bz2 commented Oct 10, 2015

(Review request: http://reviews.vapour.ws/r/2872/)
@wallyworld
Owner

wallyworld commented Oct 10, 2015

LGTM
@bz2
Contributor

bz2 commented Oct 10, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Oct 10, 2015

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

jujubot commented Oct 10, 2015

Build failed: Merging failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-cmd/1
@bz2
Contributor

bz2 commented Oct 11, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Oct 11, 2015

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

jujubot added a commit that referenced this pull request Oct 11, 2015

@jujubot
Merge pull request #22 from bz2/use_yaml_v2 
Switch to using gopkg.in/yaml.v2



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

@jujubot jujubot merged commit e233fd7 into juju:master Oct 11, 2015

@davecheney davecheney referenced this pull request Oct 20, 2015

Merged

Revert "Switch to using gopkg.in/yaml.v2" #25

jujubot added a commit that referenced this pull request Oct 20, 2015

@jujubot
Merge pull request #25 from davecheney/revert-22-use_yaml_v2 
Revert "Switch to using gopkg.in/yaml.v2"

Reverts juju/cmd#22

Breaks github.com/juju/juju/cmd/juju/status, the handling of empty fields is different between these versions.

(Review request: http://reviews.vapour.ws/r/2945/)
081dd27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment