/juju

Add PR #241 for goyaml now until it gets merged upstream. #6983

Merged
merged 2 commits into from Feb 15, 2017

Conversation

Reviewers

@wallyworld wallyworld

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@howbazaar @jujubot @wallyworld
@howbazaar
Owner

howbazaar commented Feb 15, 2017

In order to better support the YAML spec, values that have leading zeros are strings, not numbers.

QA steps

From the bug, create a charm with the following config option:

    type: string
    description: The key for the Debian package repository.
    default: 01182252

Deploy it, check the config for the app.

Bug reference

Fixes lp:1543660
@howbazaar
Owner

howbazaar commented Feb 15, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Feb 15, 2017

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

jujubot commented Feb 15, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10291
@howbazaar
Owner

howbazaar commented Feb 15, 2017

$$merge$$
@jujubot
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 67a76c9 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