/charm

Clarified error message (2) #187

Merged
merged 2 commits into from Jan 6, 2016

Conversation

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

dooferlad commented Jan 6, 2016

No description provided.

bundledata.go
@@ -473,8 +473,8 @@ func (verifier *bundleDataVerifier) verifyEndpointBindings() {
if !(matchedProvides || matchedRequires || matchedPeers) {
verifier.addErrorf(
- "service %s wants to bind to endpoint %s to space %s, "+
@dimitern

dimitern Jan 6, 2016

LGTM with s/%s/%q/ in the message, please.

@dooferlad
Contributor

dooferlad commented Jan 6, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Jan 6, 2016

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

jujubot added a commit that referenced this pull request Jan 6, 2016

@jujubot
Merge pull request #187 from dooferlad/bundle-endpoint-bindings 
Clarified error message (2)
0d22d54

@jujubot jujubot merged commit 0d22d54 into juju:v6-unstable Jan 6, 2016

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