/cmd

Remove WriterFactory #24

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

Remove the WriterFactory interface and replace it with a function.

Simpler for the caller to implement, and we only have one caller, cmd/jujud/

(Review request: http://reviews.vapour.ws/r/2943/)
@davecheney
Remove WriterFactory 
Remove the WriterFactory interface and replace it with a function.
ae6829f
@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-cmd

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

@jujubot
Merge pull request #24 from davecheney/writer-factory 
Remove WriterFactory

Remove the WriterFactory interface and replace it with a function.

Simpler for the caller to implement, and we only have one caller, cmd/jujud/

(Review request: http://reviews.vapour.ws/r/2943/)
c3d54bf

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

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