/juju

cmd/jujud/agent: drive by test gardening #5607

Merged
merged 1 commit into from Jun 14, 2016

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 Jun 14, 2016 •
Edited 1 time

YAGNI a bit of overengineering discovered while chasing workers that do
not stop.

(Review request: http://reviews.vapour.ws/r/5050/)
@davecheney
cmd/jujud/agent: drive by test gardening 
YAGNI a bit of overengineering discovered while chasing workers that do
not stop.
8da31f3
@davecheney
Contributor

davecheney commented Jun 14, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Jun 14, 2016

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

jujubot commented Jun 14, 2016

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

davecheney commented Jun 14, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Jun 14, 2016

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

@jujubot jujubot merged commit 5f9c4ea into juju:master Jun 14, 2016

@davecheney davecheney deleted the davecheney:yagni branch Jun 14, 2016

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