search

tryfer / node-tryfer

A Node Zipkin Tracer Library.
Apache License 2.0
44 stars 27 forks source link

use node-microtime module for timestamps #18

Closed wadey closed 10 years ago

wadey commented 10 years ago

add node-microtime as an optionalDependency, and use it for microsecond timestamps instead of Date.now() * 1000. If the dependency cannot be installed (since it is a native module) fail back to the old method.

https://github.com/wadey/node-microtime

cyli commented 10 years ago

@wadey Thank you! This is great! It's missing some semicolons, which is why it's not linting. I've fixed it in a different PR, and will merge that one. Should I then make a new release?

wadey commented 10 years ago

:+1: looks good to me, sorry forgot to run the linter.