/utils

Backport {Read,Write}Yaml improvements from juju/juju/utils #172

Merged
merged 1 commit into from Nov 9, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@davecheney @jujubot
@davecheney
Contributor

davecheney commented Nov 9, 2015

  • Backport improved {Read,Write}Yaml functions forked to juju/juju/utils
  • Add tests to check implicit invariants

When the yaml.v1, v2 migration is complete, the version in juju/juju/utils
will be deleted.

(Review request: http://reviews.vapour.ws/r/3089/)
@davecheney
Backport {Read,Write}Yaml improvements from juju/juju/utils 
* Backport improved {Read,Write}Yaml functions forked to juju/juju/utils
* Add tests to check implicit invariants

When the yaml.v1, v2 migration is complete, the version in juju/juju/utils
will be deleted.
3ab3e7e
@davecheney
Contributor

davecheney commented Nov 9, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Nov 9, 2015

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

jujubot added a commit that referenced this pull request Nov 9, 2015

@jujubot
Merge pull request #172 from davecheney/add-yaml-tests 
Backport {Read,Write}Yaml improvements from juju/juju/utils

* Backport improved {Read,Write}Yaml functions forked to juju/juju/utils
* Add tests to check implicit invariants

When the yaml.v1, v2 migration is complete, the version in juju/juju/utils
will be deleted.

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

@jujubot jujubot merged commit cfaebdd into juju:master Nov 9, 2015

@davecheney davecheney deleted the davecheney:add-yaml-tests branch Nov 9, 2015

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