/juju

cmd/jujud: simplify cmd.Log factory #3545

Merged
merged 1 commit into from Oct 20, 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 Oct 20, 2015

Update to juju/cmd c3d54bf5.

Simplify logic for installing a cmd logger factory.

(Review request: http://reviews.vapour.ws/r/2944/)
@davecheney
Contributor

davecheney commented Oct 20, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Oct 20, 2015

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

jujubot commented Oct 20, 2015

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/5149
@davecheney
Contributor

davecheney commented Oct 20, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Oct 20, 2015

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

jujubot commented Oct 20, 2015

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/5150
@davecheney
Contributor

davecheney commented Oct 20, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Oct 20, 2015

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

jujubot commented Oct 20, 2015

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/5151
@davecheney
cmd/jujud: simplify cmd.Log factory 
Update to juju/cmd c3d54bf5.

Simplify logic for installing a cmd logger factory.
9d53ae7
@davecheney
Contributor

davecheney commented Oct 20, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Oct 20, 2015

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

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

@jujubot
Merge pull request #3545 from davecheney/remove-writer-factory 
cmd/jujud: simplify cmd.Log factory

Update to juju/cmd c3d54bf5.

Simplify logic for installing a cmd logger factory.

(Review request: http://reviews.vapour.ws/r/2944/)
719cb3b

@jujubot jujubot merged commit 719cb3b into juju:master Oct 20, 2015

@davecheney davecheney deleted the davecheney:remove-writer-factory branch Oct 20, 2015

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