Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clock not defined? #1

Closed
Meekohi opened this issue Apr 25, 2013 · 2 comments
Closed

clock not defined? #1

Meekohi opened this issue Apr 25, 2013 · 2 comments
Assignees

Comments

@Meekohi
Copy link

Meekohi commented Apr 25, 2013

/node_modules/librato-node/lib/worker.js:55
      earliest = new Date(clock.now() + period);

ReferenceError: clock is not defined
    at Function.Worker.startTime (node_modules/librato-node/lib/worker.js:55:27)
    at Worker.start (node_modules/librato-node/lib/worker.js:25:24)
    at EventEmitter.librato.start (node_modules/librato-node/lib/librato.js:37:19)
    at Object.<anonymous> (web.js:38:9)
    at Module._compile (module.js:449:26)
    at Object.Module._extensions..js (module.js:467:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.runMain (module.js:492:10)
    at process.startup.processNextTick.process._tickCallback (node.js:244:9)
@ghost ghost assigned bobzoller Apr 25, 2013
@bobzoller
Copy link
Contributor

Thanks for the report!

I just pushed + published v0.0.5 that should address this. clock was a reference to node-clock, which is a global in the project I extracted librato-node out of... Replaced them with Date.now() instead of adding node-clock, since this doesn't need any of that functionality.

@hazeledmands
Copy link
Contributor

Closing -- pretty sure this was resolved long ago

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants