/utils

utils/clock: add clock package #147

Merged
merged 1 commit into from Aug 10, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@axw @wallyworld @jujubot
@axw
Member

axw commented Aug 10, 2015

Add a clock package which contains a Clock
interface, and WallClock var whose type
implements Clock in terms of the wall clock.

(Review request: http://reviews.vapour.ws/r/2327/)
clock/wall.go
+
+// Alarm returns a channel that will send a value at some point after
+// the supplied time.
+func (clock wallClock) After(d time.Duration) <-chan time.Time {
@wallyworld

wallyworld Aug 10, 2015

Owner

don't need a named receiver

@axw

axw Aug 10, 2015

Member

fixed, ta

@wallyworld
Owner

wallyworld commented Aug 10, 2015

LGTM
@axw
utils/clock: add clock package 
Add a clock package which contains a Clock
interface, and WallClock var whose type
implements Clock in terms of the wall clock.
653f297
@axw
Member

axw commented Aug 10, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Aug 10, 2015

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

jujubot added a commit that referenced this pull request Aug 10, 2015

@jujubot
Merge pull request #147 from axw/juju-utils-clock 
utils/clock: add clock package

Add a clock package which contains a Clock
interface, and WallClock var whose type
implements Clock in terms of the wall clock.

(Review request: http://reviews.vapour.ws/r/2327/)
6f48322

@jujubot jujubot merged commit 6f48322 into juju:master Aug 10, 2015

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