/charm

Update test to match changed juju/version behaviour #219

Merged
merged 1 commit into from Sep 15, 2016

Conversation

Reviewers

@natefinch natefinch

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@babbageclunk @jujubot @natefinch
@babbageclunk
Member

babbageclunk commented Sep 15, 2016

Since commit 4ae6172 (juju/version#3), version.Number.Tag will never include any digits, so this test became incorrect.
@babbageclunk
Update test to match changed juju/version behaviour 
Since commit 4ae6172 (juju/version#3),
version.Number.Tag will never include any digits, so this test was
incorrect.
340c221
@natefinch

natefinch approved these changes Sep 15, 2016
View changes

LGTM

@babbageclunk
Member

babbageclunk commented Sep 15, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Sep 15, 2016

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

@jujubot jujubot merged commit 2f2476d into juju:v6-unstable Sep 15, 2016

@babbageclunk babbageclunk deleted the babbageclunk:fix-test branch Sep 15, 2016

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