search

mikejihbe / metrics

A metrics library for Node.js
574 stars 58 forks source link

Gauge and CachedGauge introduced. #59

Closed rafalsiwiec closed 5 years ago

rafalsiwiec commented 5 years ago

Implements #36

tolbertam commented 5 years ago

You probably already had this in mind but this should also update all the reporter implementations to report the gauges as well.

rafalsiwiec commented 5 years ago

@tolbertam please check this out.

rafalsiwiec commented 5 years ago

@tolbertam any chance to review it soon?

tolbertam commented 5 years ago

Sorry for the delay, have been busy lately. I'll take a look at it this week :)

mistic100 commented 5 years ago

@tolbertam did you had time to take a look ? :stuck_out_tongue_closed_eyes:

tolbertam commented 5 years ago

terribly sorry, I've had a lot going on and just haven't had much time. I'll try to set aside some time tonight to work on this.

mistic100 commented 5 years ago

No pressure :)

tolbertam commented 5 years ago

It looks like one of our dependencies now requires ES6, so travis runs against Node.js <= 4 no longer work, i'll fix this in a separate PR.